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

[16.01] Update help text rST note/warning/error box styling for change in div class from toolHelpBody to ui-form-help. #1695

Merged
merged 1 commit into from
Feb 8, 2016

Conversation

natefoo
Copy link
Member

@natefoo natefoo commented Feb 8, 2016

Fixes the regression pointed out by @bgruening in #685

@natefoo natefoo added this to the 16.01 milestone Feb 8, 2016
martenson added a commit that referenced this pull request Feb 8, 2016
[16.01] Update help text rST note/warning/error box styling for change in div class from toolHelpBody to ui-form-help.
@martenson martenson merged commit 054c04f into galaxyproject:release_16.01 Feb 8, 2016
@martenson
Copy link
Member

@natefoo changes in base.less should only result in additional changes in base.css so I assume that the additional JS files are forgotten client build

@natefoo
Copy link
Member Author

natefoo commented Feb 8, 2016

I was wondering about that as well, but it looks like these specific files update more frequently (look at their histories). If they're pulled from external sources that would explain it? Sorry, I don't know enough about the client code to comment intelligently here.

@dannon
Copy link
Member

dannon commented Feb 8, 2016

Differing dependency versions have been known to pack/bundle the files slightly differently in the past, and that could explain it as well.

@martenson
Copy link
Member

@dannon but not functionally differently I hope?

@dannon
Copy link
Member

dannon commented Feb 8, 2016

@martenson Yay github email delay :)

No, they still both seemed to work fine, just slightly different packing algorithms.

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

Successfully merging this pull request may close these issues.

3 participants