User tests: Successful: Unsuccessful:
Pull Request for Issue #36117 .
Added Underline in the link in Joomla Update Section.
See the link underlined in the Joomla update section and set the colour to monochrome for atum and see the link (underlined).
No underlined link.
Underlined link in Joomla Update Section.
No
Status | New | ⇒ | Pending |
Category | ⇒ | Administration Templates (admin) NPM Change |
Sorry my mistake. Closed by accident.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-11-26 22:15:34 |
Closed_By | ⇒ | richard67 | |
Labels |
Added:
?
NPM Resource Changed
|
Status | Closed | ⇒ | New |
Closed_Date | 2021-11-26 22:15:34 | ⇒ | |
Closed_By | richard67 | ⇒ |
Status | New | ⇒ | Pending |
I have tested this item
with prebuilt package
I think this PR has some unwanted side effects. It underlines EVERY link in monochrome mode, even the icons.
See the red marks in the following screen shot:
@chmst Is this what you had in mind when reporting the issue? Or did you just refer to the link at the bottom?
I think it needs to be more specific here with the CSS selector.
Fairly certain that the monochrome screenshot was to emphasise the problem of that link and not that the change should be made just for monochrome
@brianteeman Would you say it justifies a negative test in the tracker? Or is it more caused by a probably misleading description of the issue?
It's a negative test because of the issue you reported.
It's not the correct solution if my assumption is correct
@brianteeman I think we think the same: That one link should be underlined regardless of monochrome or not because without monochrome the only visual indicator for it being a link is the colour, and that's not enough. Other links should not be touched if not having the same issue.
I have tested this item
With the PR not only that one link is underlined but also other links and even the icons for these links, see the red marks in the screen shot in my comment above #36134 (comment) .
I think a better fix would be to make sure that the one link which has no visual indication at all when monochrome and only the colour when not monochrome should get underlined in any case, regardless of monochrome or not, and the other links should not be touched if they don't have the same problem as thins one link has.
Links looking like buttons and icons attached to the link text should never be underlined in my opinion because that simply looks ugly and has no benefit.
@richard67 @brianteeman Thanks for the feedback. I will do the required changes as suggested and will update the PR accordingly.
I have not tested this item.
sorry did not pay attention to the other underlined links.
Category | Administration Templates (admin) NPM Change | ⇒ | Administration com_joomlaupdate |
I have committed the desired changes. Thanks :)
Now after the last change it needs to update title and description because they still refer to monochrome.
Title |
|
@richard67 Thanks for the suggestion. I have updated the title and description of the PR.
I have tested this item
Yes., but as much as I know there are no other links on "Empty state screens". The update process is something "extra" and it seems overkill now to find generic solutions here.
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
I have tested this item
Labels |
Added:
?
Removed: NPM Resource Changed |
Labels |
Added:
?
Removed: ? |
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-01-31 17:40:00 |
Closed_By | ⇒ | bembelimen |
Thx
Please fix the scss code style errors reported here https://ci.joomla.org/joomla/joomla-cms/48730/1/20 . Thanks in advance.