This repository has been archived by the owner on Jun 7, 2023. It is now read-only.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… making CategorySelect into a controller.
Split hooks into CategorySelectHooksHelper class, split html into templates, created CategorySelectController to handle requests.
…e, also set up skeleton for the other functions.
Remove the 'source mode' requirements of maintaining a wikitext data structure. Use the same UI and underlying data structures for both visual and source modes (BugId:48693)."
Added JavaScript logic for adding categories. Made the behavior of accessing categories more consistent. Moved the normalizing category function into the CategorySelect class.
Also create getExtractedCategoryData as a getter instead of trying to use getData as a multipurposed function (it's more explicit this way).
…change summary Conflicts: extensions/wikia/EditPageLayout/js/editor/WikiaEditorStorage.js
…ySelectStateOnLogin Bugid 95455 wikia editor save category select state on login
Conflicts: extensions/wikia/CategorySelect/CategorySelect.i18n.php extensions/wikia/Development/Configs_DevBox/LocalSettings.php extensions/wikia/EditPageLayout/js/editor/WikiaEditorStorage.js
…mplate and into controller.
Fix Sizzle exceptions for invalid expressions when trying to match against strings in getCategories. Always uppercase the first letter of a category to properly match Mediawiki titles.
Alright, I think this is good to go. Going to get a design / product check then we'll see about the merge. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a rather large commit, so here are the gist of the changes: