Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme: Webmin API ui_form_end buttons not working #543

Closed
KramNotlimah opened this issue Aug 31, 2016 · 6 comments
Closed

Theme: Webmin API ui_form_end buttons not working #543

KramNotlimah opened this issue Aug 31, 2016 · 6 comments

Comments

@KramNotlimah
Copy link

KramNotlimah commented Aug 31, 2016

I am writing a module for Virtualmin and I ran across an issue with the ui_form_end subroutine in the Webmin core API. If you open a popup window and use the popup_header instead of the ui_print_header the buttons all come out with a value="OK". In the original Virtualmin theme with the same code they came out with the correct values. When using the ui_print_header in the main windows the ui_form_end produces buttons fine in the original Virtualmin theme and in the Authentic Theme. It seems it is only an issue when using the popup_header with Authentic. I worked around the issue by just creating my own buttons but it would be nice to be able to use the API as advertised.

Mark

@iliajie
Copy link
Collaborator

iliajie commented Aug 31, 2016

Hi, Mark.

I will run tests a bit later and try fixing it upon next release.

@iliajie iliajie changed the title Webmin API ui_form_end buttons not working with Authentic Theme. Theme: Webmin API ui_form_end buttons not working Aug 31, 2016
@KramNotlimah
Copy link
Author

Thanks. That sounds great.

Mark

@iliajie
Copy link
Collaborator

iliajie commented Oct 16, 2016

Mark, where could I test it in VM or WM?

@iliajie
Copy link
Collaborator

iliajie commented Oct 16, 2016

Ahh, ok, got it! It will be fixed in 18.20.

@iliajie iliajie closed this as completed Oct 16, 2016
@iliajie
Copy link
Collaborator

iliajie commented Oct 29, 2016

Version 18.20 is out, please upgrade.

@KramNotlimah
Copy link
Author

Thanks! I will get the update.

Mark

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants