NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar himanshu007-creator
himanshu007-creator
23 May 2021

Pull Request for Issue #34109 .

Summary of Changes

On enabling Highlight Links in accessibility, there was mismatch in size of toolbar items because of additional padding.
But keeping accessibility in mind, other things should be left unchanged, therefore just removed the padding.

Testing Instructions

  • apply the patch and rebuild css npm run build:css
  • turn highlight links on from accessibility and note, size doesnt change

Actual result BEFORE applying this Pull Request

old

Expected result AFTER applying this Pull Request

ssss

Documentation Changes Required

avatar himanshu007-creator himanshu007-creator - open - 23 May 2021
avatar himanshu007-creator himanshu007-creator - change - 23 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 May 2021
Category Administration Templates (admin) NPM Change
1c418e7 23 May 2021 avatar himanshu007-creator lint
avatar himanshu007-creator himanshu007-creator - change - 23 May 2021
Labels Added: NPM Resource Changed ?
avatar PhilETaylor
PhilETaylor - comment - 23 May 2021

I think you misunderstood the issue.

The issue is

  1. the User Menu oval background is not the same height and depth as the other two (Postinstallation messages and joomla4 in your screenshot)

  2. User Menu is not underlined.

Screenshot 2021-05-23 at 23 18 58

  1. User Menu has no border/highlight
avatar himanshu007-creator himanshu007-creator - change - 24 May 2021
The description was changed
avatar himanshu007-creator himanshu007-creator - edited - 24 May 2021
avatar himanshu007-creator
himanshu007-creator - comment - 25 May 2021

@PhilETaylor made the changes as suggested by you.

avatar PhilETaylor
PhilETaylor - comment - 25 May 2021

Looks good - thanks.

Screenshot 2021-05-25 at 20 06 57

Screenshot 2021-05-25 at 20 07 45

avatar PhilETaylor PhilETaylor - test_item - 25 May 2021 - Tested successfully
avatar PhilETaylor
PhilETaylor - comment - 25 May 2021

I have tested this item successfully on f7dedbc


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

avatar sandramay0905 sandramay0905 - test_item - 26 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 26 May 2021

I have tested this item successfully on f7dedbc


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

avatar richard67 richard67 - change - 26 May 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 26 May 2021

RTC


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

avatar richard67 richard67 - change - 26 May 2021
Status Ready to Commit Pending
avatar richard67
richard67 - comment - 26 May 2021

Back to pending. @himanshu007-creator Could you implement @Quy 's suggestion? Thanks in advance.


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

avatar himanshu007-creator himanshu007-creator - change - 26 May 2021
Labels Added: ?
avatar himanshu007-creator
himanshu007-creator - comment - 26 May 2021

Made the changes @richard67

avatar himanshu007-creator himanshu007-creator - change - 26 May 2021
Labels Removed: ?
avatar sandramay0905 sandramay0905 - test_item - 27 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 27 May 2021

I have tested this item successfully on 9d2c630

After update J4 using custom update server of prebuild package:

image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34146.
avatar srishty-07 srishty-07 - test_item - 27 May 2021 - Tested successfully
avatar srishty-07
srishty-07 - comment - 27 May 2021

I have tested this item successfully on 9d2c630


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

avatar Quy Quy - change - 27 May 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-27 13:04:22
Closed_By Quy
avatar Quy Quy - close - 27 May 2021
avatar Quy Quy - merge - 27 May 2021
avatar Quy
Quy - comment - 27 May 2021

Thanks!

Add a Comment

Login with GitHub to post a comment