Skip to content

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Dec 22, 2017

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.

Updated the label and description as per @ciar4n request

Pull Request for Issue #19122

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.

Updated the label and description as per @ciar4n request
@Quy
Copy link
Contributor

Quy commented Dec 23, 2017

I have tested this item ✅ successfully on c00a60a


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

1 similar comment
@ghost
Copy link

ghost commented Dec 23, 2017

I have tested this item ✅ successfully on c00a60a


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

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-staging labels Dec 23, 2017
@ghost
Copy link

ghost commented Dec 23, 2017

Ready to Commit after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 23, 2017
COM_MENUS_ITEM_FIELD_ALIAS_MENU_LABEL="Menu Item"
COM_MENUS_ITEM_FIELD_ANCHOR_CSS_DESC="An optional, custom style to apply to the menu hyperlink."
COM_MENUS_ITEM_FIELD_ANCHOR_CSS_LABEL="Link CSS Style"
COM_MENUS_ITEM_FIELD_ANCHOR_CSS_DESC="An optional class to apply to the menu hyperlink."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

COM_MENUS_ITEM_FIELD_ANCHOR_CSS_DESC="An optional CSS class to apply to the menu hyperlink."

I think it will be more understandable 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As pointed out by @ciar4n in the original issue it could be JavaScript.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging and removed RTC This Pull Request is Ready To Commit labels Dec 23, 2017
@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the _Link_CSS_Style branch December 23, 2017 16:18
photodude pushed a commit to photodude/joomla-cms that referenced this pull request Dec 25, 2017
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.

Updated the label and description as per @ciar4n request
zero-24 added a commit to zero-24/joomla that referenced this pull request Jan 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language Change This is for Translators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants