? ? Pending

User tests: Successful: Unsuccessful:

avatar khu5h1
khu5h1
26 Nov 2021

Pull Request for Issue #36117 .

Summary of Changes

Added Underline in the link in Joomla Update Section.

Testing Instructions

See the link underlined in the Joomla update section and set the colour to monochrome for atum and see the link (underlined).

Actual result BEFORE applying this Pull Request

No underlined link.

Expected result AFTER applying this Pull Request

Underlined link in Joomla Update Section.

Documentation Changes Required

No

avatar khu5h1 khu5h1 - open - 26 Nov 2021
avatar khu5h1 khu5h1 - change - 26 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Nov 2021
Category Administration Templates (admin) NPM Change
avatar richard67
richard67 - comment - 26 Nov 2021

Please fix the scss code style errors reported here https://ci.joomla.org/joomla/joomla-cms/48730/1/20 . Thanks in advance.

avatar richard67
richard67 - comment - 26 Nov 2021

Sorry my mistake. Closed by accident.

avatar richard67 richard67 - change - 26 Nov 2021
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
avatar richard67 richard67 - close - 26 Nov 2021
avatar richard67 richard67 - change - 26 Nov 2021
Status Closed New
Closed_Date 2021-11-26 22:15:34
Closed_By richard67
avatar richard67 richard67 - change - 26 Nov 2021
Status New Pending
avatar richard67 richard67 - reopen - 26 Nov 2021
avatar ChristineWk
ChristineWk - comment - 27 Nov 2021

I have tested this item successfully on 458a474

with prebuilt package


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar ChristineWk ChristineWk - test_item - 27 Nov 2021 - Tested successfully
avatar richard67
richard67 - comment - 27 Nov 2021

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:

2021-11-27_monochrome

@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.

avatar brianteeman
brianteeman - comment - 27 Nov 2021

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

avatar richard67
richard67 - comment - 28 Nov 2021

@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?

avatar brianteeman
brianteeman - comment - 28 Nov 2021

It's a negative test because of the issue you reported.
It's not the correct solution if my assumption is correct

avatar richard67
richard67 - comment - 28 Nov 2021

@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.

avatar richard67 richard67 - test_item - 28 Nov 2021 - Tested unsuccessfully
avatar richard67
richard67 - comment - 28 Nov 2021

I have tested this item ? unsuccessfully on 458a474

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.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar khu5h1
khu5h1 - comment - 28 Nov 2021

@richard67 @brianteeman Thanks for the feedback. I will do the required changes as suggested and will update the PR accordingly.

avatar ChristineWk
ChristineWk - comment - 28 Nov 2021

I have not tested this item.

sorry did not pay attention to the other underlined links.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar ChristineWk ChristineWk - test_item - 28 Nov 2021 - Not tested
avatar chmst
chmst - comment - 28 Nov 2021

Sorry, was absent for a few days. Thank you @khu5h1 for working on that issue. I was surprised myself by this screen as in general we did carefully underline links.
Seems that the new empty-state texts were forgotten.

avatar joomla-cms-bot joomla-cms-bot - change - 29 Nov 2021
Category Administration Templates (admin) NPM Change Administration com_joomlaupdate
avatar khu5h1
khu5h1 - comment - 29 Nov 2021

I have committed the desired changes. Thanks :)

avatar richard67
richard67 - comment - 29 Nov 2021

Now after the last change it needs to update title and description because they still refer to monochrome.

avatar khu5h1 khu5h1 - change - 30 Nov 2021
Title
Fixed#36117 -- Underlined links in monochrome atum theme
Fixed#36117 -- Underlined links in Joomla Update Section
avatar khu5h1 khu5h1 - edited - 30 Nov 2021
avatar khu5h1 khu5h1 - change - 30 Nov 2021
The description was changed
avatar khu5h1 khu5h1 - edited - 30 Nov 2021
avatar khu5h1
khu5h1 - comment - 30 Nov 2021

@richard67 Thanks for the suggestion. I have updated the title and description of the PR.

avatar chmst
chmst - comment - 2 Dec 2021

I have tested this item successfully on 4117a51


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar chmst chmst - test_item - 2 Dec 2021 - Tested successfully
avatar chmst
chmst - comment - 2 Dec 2021

@khu5h1
thanks for your contribution.
Underlining the link is correct independent from monochrome style. It was an accessibility issue as colorblind users see every screen as monochrome.

avatar richard67
richard67 - comment - 2 Dec 2021

@chmst Shouldn't that be done in other places, too, where we have a link at the bottom e.g. of a blank state page?

avatar chmst
chmst - comment - 2 Dec 2021

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.

avatar ChristineWk
ChristineWk - comment - 2 Dec 2021

I have tested this item successfully on 4117a51


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar ChristineWk ChristineWk - test_item - 2 Dec 2021 - Tested successfully
avatar richard67 richard67 - change - 2 Dec 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 2 Dec 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar pritam825
pritam825 - comment - 4 Dec 2021

I have tested this item successfully on 4117a51


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36134.

avatar pritam825 pritam825 - test_item - 4 Dec 2021 - Tested successfully
avatar Quy Quy - change - 27 Jan 2022
Labels Added: ?
Removed: NPM Resource Changed
avatar chmst chmst - change - 30 Jan 2022
Labels Added: ?
Removed: ?
avatar bembelimen bembelimen - close - 31 Jan 2022
avatar bembelimen bembelimen - merge - 31 Jan 2022
avatar bembelimen bembelimen - change - 31 Jan 2022
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
avatar bembelimen
bembelimen - comment - 31 Jan 2022

Thx

Add a Comment

Login with GitHub to post a comment