Skip to content

Comments

[4.0] table head font weight#32958

Merged
Quy merged 1 commit intojoomla:4.0-devfrom
brianteeman:bugsme
Apr 2, 2021
Merged

[4.0] table head font weight#32958
Quy merged 1 commit intojoomla:4.0-devfrom
brianteeman:bugsme

Conversation

@brianteeman
Copy link
Contributor

Maybe its just me but I find it very unusual that the cells in the table head are not "bold" which is pretty normal.

It's not that obvious on a page like the article manager as all the cells are links but its really noticeable in the modules on the dashboard and in lists where not everything is a link eg access levels

On the assumption that its not just me here is a pr for consideration. As with all scss changes you will need to npm to build the css

Before

image

image

image

After

image

image

image

Maybe its just me but I find it very unusual that the cells in the table head are not "bold" which is pretty normal.

It's not that obvious on a page like the article manager as all the cells are links but its really noticeable in the modules on the dashboard and in lists where not everything is a link eg access levels

On the assumption that its not just me here is a pr for consideration. As with all scss changes you will need to npm to build the css
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Apr 1, 2021
@ceford
Copy link
Contributor

ceford commented Apr 2, 2021

I have tested this item ✅ successfully on d4405a4

Me too! I think this just reverts to browser default style (good).


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

@ChristineWk
Copy link

I have tested this item ✅ successfully on d4405a4


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

@Quy Quy removed the NPM Resource Changed This Pull Request can't be tested by Patchtester label Apr 2, 2021
@Quy Quy added this to the Joomla 4.0 milestone Apr 2, 2021
@Quy Quy merged commit 3fdb98e into joomla:4.0-dev Apr 2, 2021
@joomla-cms-bot joomla-cms-bot added the NPM Resource Changed This Pull Request can't be tested by Patchtester label Apr 2, 2021
@Quy
Copy link
Contributor

Quy commented Apr 2, 2021

Thanks

@brianteeman
Copy link
Contributor Author

thanks

@brianteeman brianteeman deleted the bugsme branch April 2, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants