?
avatar ciar4n
ciar4n
21 Dec 2017

Steps to reproduce the issue

Navigate to the Link Type tab of a menu item.

The Link CSS Style field label should be Link Class. The field adds a class to the menu item link, the current label implies it is adding inline CSS.

avatar ciar4n ciar4n - open - 21 Dec 2017
avatar joomla-cms-bot joomla-cms-bot - change - 21 Dec 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Dec 2017
avatar ciar4n ciar4n - change - 21 Dec 2017
The description was changed
avatar ciar4n ciar4n - edited - 21 Dec 2017
avatar ciar4n ciar4n - change - 21 Dec 2017
The description was changed
avatar ciar4n ciar4n - edited - 21 Dec 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Dec 2017
Category Layout
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Dec 2017
Category Layout Layout Templates (admin) UI/UX
avatar brianteeman
brianteeman - comment - 21 Dec 2017

Links CSS Class ? perhaps

avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Dec 2017
Status New Discussion
avatar ciar4n
ciar4n - comment - 21 Dec 2017

IMO still not correct as a class can also be used for JS. It is also not consistent with other class fields (eg. Module Class Suffix, Header Class).

avatar brianteeman
brianteeman - comment - 21 Dec 2017

IMO still not correct as a class can also be used for JS.

thats exactly why i suggested adding the word CSS

avatar ciar4n
ciar4n - comment - 21 Dec 2017

Why add 'CSS' when it can also be used for JS?

avatar brianteeman
brianteeman - comment - 21 Dec 2017

ah - i thought you could only reference a css class there

avatar ciar4n
ciar4n - comment - 21 Dec 2017

In most cases I suspect it is used with CSS but it can potentially have a JS use. My 2 cents would be to change it to simply Link Class but happy enough with either.

avatar brianteeman
brianteeman - comment - 21 Dec 2017

For consistency then I guess Link Class is fine

avatar brianteeman brianteeman - change - 22 Dec 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-12-22 22:41:58
Closed_By brianteeman
avatar brianteeman brianteeman - close - 22 Dec 2017
avatar brianteeman
brianteeman - comment - 22 Dec 2017

closed see #19143

Add a Comment

Login with GitHub to post a comment