Branch: refs/heads/master
Home: https://github.com/xwiki/xwiki-platform
Commit: 54932a8ee1fd36d7ed5dc8d95ea0634aeebe39e9
https://github.com/xwiki/xwiki-platform/commit/54932a8ee1fd36d7ed5dc8d95ea0…
Author: LucasC <lucas.charpentier(a)imt-atlantique.net>
Date: 2024-02-29 (Thu, 29 Feb 2024)
Changed paths:
M xwiki-platform-core/xwiki-platform-search/xwiki-platform-search-ui/src/main/resources/XWiki/SearchSuggestSourceSheet.xml
M xwiki-platform-distribution/xwiki-platform-distribution-flavor/xwiki-platform-distribution-flavor-test/pom.xml
Log Message:
-----------
XWIKI-21779: Admin section: make the search suggest section pass webstandard tests (#2816)
* XWIKI-21779: Admin section: make the search suggest section pass webstandard tests
* Updated the template to avoid duplicating IDs
Somehow adding the empty object as a parameter changes the ID (adds a {} at the start, still a correct ID, and different from the one used in Config), but does not change how the template is used to create new sources. New sources are still correctly generated.
* XWIKI-21779: Admin section: make the search suggest section pass webstandard tests
* Updated the template to avoid incorrect checkbox in a label.
There was the error `ERROR: Any “select” descendant of a “label” element with a “for” attribute must have an ID value that matches that “for” attribute. at line [1044] column [105]` which failed for the `activated` property. The previous fix did not solve that so instead I decided to put the checkbox input before the label so that it doesn't fail if it's not properly linked.
* Removed the `for` attribute for the labels in the template, because they didn't point towards any input.
* Added an empty dl after checkboxes, in order to keep a correct dd structure.
* Removed the comment from the test for searchSuggest
Note: The best solution would be to replace completely this dd with a regular list, but this is way faster to patch it.
* XWIKI-21779: Admin section: make the search suggest section pass webstandard tests
* Added a comment with a reference to the discussion on the PR
as far as I know, the PRs are kept in storage by GitHub as long as the repository is up, so adding such a link shouldn't pose problem for long term maintenance :)
To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications
Branch: refs/heads/master
Home: https://github.com/xwiki/xwiki-platform
Commit: 3707ae31211e2524969cca89808dbe1ae5a044b5
https://github.com/xwiki/xwiki-platform/commit/3707ae31211e2524969cca89808d…
Author: LucasC <lucas.charpentier(a)imt-atlantique.net>
Date: 2024-02-29 (Thu, 29 Feb 2024)
Changed paths:
M xwiki-platform-core/xwiki-platform-administration/xwiki-platform-administration-ui/src/main/resources/XWiki/AdminSheet.xml
M xwiki-platform-core/xwiki-platform-flamingo/xwiki-platform-flamingo-skin/xwiki-platform-flamingo-skin-resources/src/main/resources/flamingo/rightsUI.vm
M xwiki-platform-core/xwiki-platform-web/xwiki-platform-web-war/src/main/webapp/resources/js/xwiki/usersandgroups/usersandgroups.css
M xwiki-platform-core/xwiki-platform-web/xwiki-platform-web-war/src/main/webapp/resources/js/xwiki/usersandgroups/usersandgroups.js
Log Message:
-----------
XWIKI-21597: Make the rights UI use icon themes (#2649)
* Updated icons in the rights UI
* Removed the logic to update the source for the checkbox image
* Removed velocity calls from inside the javascript
* Removed the alt - didn't add an alternative because what it used to be is already supported by the default checkbox behaviour
* Updated the javascript to fit this change in marking even when checking/unchecking the boxes.
To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications