[4.0] com_finder ul instead of dl for easier styling#30534
[4.0] com_finder ul instead of dl for easier styling#30534wilsonge merged 4 commits intojoomla:4.0-devfrom hans2103:4.0-dev-a11y--com_finder-ol-instead-of-dl-for-easier-styling
Conversation
|
I have tested this item ✅ successfully on c9f4593 Perhaps a bit overstructured in the inner part of the li-element. There are three levels: header->h4->a for each result title and three times the same class: result-title(in li, header and h4) This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30534. |
|
Thank you for testing.
After merging this PR where DL will be replaced by UL I will create a new PR to simplify the HTML structure of a search result. I agree on the HTML structure. |
|
I have tested this item ✅ successfully on c9f4593 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30534. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30534. |
|
Makes sense. But given it’s search results I assume ol is more logical than ul as they are ordered by search relevancy |
|
@wilsonge can changing |
|
Thats true. But still think it makes more sense to do it like that |
|
Thanks! |
…om_templates * '4.0-dev' of github.com:joomla/joomla-cms: (70 commits) [4.0] Child templates consistency (joomla#30387) [4.0] favicon changes to support child templates (joomla#30388) [4.0] Update Readme for Api tests (joomla#30539) [4.0] [Multilingual Status module] Adding displaying a possible error if URL Language Code is empty (joomla#30537) [4.0] Display of horizontal mod_articles_news module (joomla#30527) [4.0] Useless installation lang strings (joomla#30568) [4.0] Numbers not digits (joomla#30559) [4.0] Accessibility plugin position (joomla#30552) [4.0] fix for inherit fields (joomla#30557) [4.0] Redundant words (joomla#30555) add missing legend to fieldset (joomla#30528) [4.0] [a11y] add statement on found results (joomla#30535) [4.0] com_finder ul instead of dl for easier styling (joomla#30534) [4.0] Messages/Alerts: using icons instead of text as heading (joomla#30516) [4.0] Increase API Test Coverage (joomla#26722) [4.0] Implementing display of password requirements for frontend (joomla#30473) [4.0] FieldsHelper: Choose a first available category correctly (joomla#30268) Sort options (joomla#30531) Clear checkboxes on back button (joomla#30498) Update _icomoon.scss (joomla#30436) ...
Pull Request for Issue # .
Summary of Changes
quote from @Hackwar taken from a private message to find out reason of changing
ultodlChanging it from DL back to UL makes it easier for frontend developers to style each search result.
Styling DT+DD is not as easy as styling a LI
Testing Instructions
blogas searched word./index.php/component/finder/search?q=blog&Itemid=101&Itemid=101
Actual result BEFORE applying this Pull Request
It is very hard to style dt+dd+dd+dd as a card. Try adding a background image, box-shadow and spacing between the individual search results.
Expected result AFTER applying this Pull Request
After applying this PR it is much easier to style individual search results.
Documentation Changes Required