Skip to content

Config: Allow different jQuery UI version to be specified for ThemeRoller page#288

Closed
rxaviers wants to merge 1 commit intomasterfrom
config-for-themeroller
Closed

Config: Allow different jQuery UI version to be specified for ThemeRoller page#288
rxaviers wants to merge 1 commit intomasterfrom
config-for-themeroller

Conversation

@rxaviers
Copy link
Member

No description provided.

@rxaviers
Copy link
Member Author

@jzaefferer @scottgonzalez , this is what I have in mind.

@jzaefferer
Copy link
Member

This still seems wrong. Can we revert the TR changes we made for 1.12 instead? Then revert-revert those once 1.12 is stable.

@rxaviers
Copy link
Member Author

ThemeRoller is wrong [1] and the goal of this PR isn't to fix it.

Having said that, I do agree with you that your suggestion is definitely a better fix than the workaround provided here. The goal of this PR is to be the "minimal updates to get 1.12.0-beta.1 announced" in a timely manner. Although, if you want to pursue for your suggestion, I can help by providing guidances. Pretty much the commit history can be used to revert the TR changes.

1: ThemeRoller should be based on the selected version, not using a static one like it currently does. FWIW, Mobile's TR has a better architecture than this one.

@scottgonzalez
Copy link
Member

@jzaefferer Are you ok with using this solution for now and implementing a real fix before 1.13?

@jzaefferer
Copy link
Member

Yeah, it'll do. We had those changes live for a while now anyway.

rxaviers added a commit that referenced this pull request Jan 20, 2016
rxaviers added a commit that referenced this pull request Jan 20, 2016
rxaviers added a commit that referenced this pull request Jan 20, 2016
@rxaviers rxaviers closed this Jan 20, 2016
@rxaviers rxaviers deleted the config-for-themeroller branch January 20, 2016 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants