NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
11 May 2021

This PR corrects multiple issues with the menu links displayed in a card on a cpanel to make the styling match the other modules on the cpanel

  1. Makes font the same size
  2. Makes the font weight the same
  3. Makes the link underlined
  4. Makes the clickable area of the link just the text not the entire row
  5. Slightly increase the size of the + button

Dont forget to rebuild the css

Thanks to @Kostelano and @HimAnshu007

before

image

after

image

avatar brianteeman brianteeman - open - 11 May 2021
avatar brianteeman brianteeman - change - 11 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 May 2021
Category Modules Administration Templates (admin) NPM Change
e525d0d 11 May 2021 avatar brianteeman b
avatar brianteeman brianteeman - change - 11 May 2021
Labels Added: NPM Resource Changed ?
avatar PhilETaylor
PhilETaylor - comment - 11 May 2021

Makes the link underlined

Might want to check out @ciar4n PR #33808

avatar PhilETaylor PhilETaylor - test_item - 11 May 2021 - Tested unsuccessfully
avatar PhilETaylor
PhilETaylor - comment - 11 May 2021

I have tested this item ? unsuccessfully on e525d0d

I think you broke System dashboard

Screenshot 2021-05-11 at 21 42 43


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33811.
avatar brianteeman
brianteeman - comment - 13 May 2021

@PhilETaylor system dashboard now fixed.

avatar sandramay0905
sandramay0905 - comment - 13 May 2021
  1. Makes the clickable area of the link just the text not the entire row

Is this correct: If there are other columns (plus-sign; hits, date) only text is clickable. If there is no other column the row is clickable (featured articles)?

image

If i understand your test instruction correct, "Featured Articles" should also only on text be clickable.

avatar brianteeman
brianteeman - comment - 13 May 2021

thats correct

avatar sandramay0905 sandramay0905 - test_item - 13 May 2021 - Tested unsuccessfully
avatar sandramay0905
sandramay0905 - comment - 13 May 2021

I have tested this item ? unsuccessfully on eb450a1

As comment above in single lines of text like "Featured Articles" the row is clickable, not only the text.


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

avatar brianteeman
brianteeman - comment - 13 May 2021

Thats the intended behaviour

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

I have tested this item successfully on eb450a1


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

avatar Kostelano Kostelano - test_item - 13 May 2021 - Tested successfully
avatar Kostelano
Kostelano - comment - 13 May 2021

I have tested this item successfully on eb450a1

PR works. The only thing that is confusing is what Phil mentioned - the problem of underlining for external links.


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

avatar brianteeman
brianteeman - comment - 13 May 2021

PR works. The only thing that is confusing is what Phil mentioned - the problem of underlining for external links.

Mentioned what? where?

avatar Kostelano
Kostelano - comment - 13 May 2021

Sorry, I'm confused. Not in this PR. But I was talking about comment #33808 (comment).

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

RTC


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

avatar Quy Quy - close - 13 May 2021
avatar Quy Quy - merge - 13 May 2021
avatar Quy Quy - change - 13 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-13 18:57:06
Closed_By Quy
Labels Added: ?
avatar Quy
Quy - comment - 13 May 2021

Thank you!

avatar brianteeman
brianteeman - comment - 13 May 2021

thx

avatar PhilETaylor
PhilETaylor - comment - 13 May 2021

Just circling back to say the + icons make so much more sense at the new size ! My eyes can actually see them now, never realised how small they were before! Thanks...

Add a Comment

Login with GitHub to post a comment