Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link CSS style strings #19143

Merged
merged 1 commit into from
Dec 23, 2017
Merged

Link CSS style strings #19143

merged 1 commit into from
Dec 23, 2017

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
@@ -80,8 +80,8 @@ COM_MENUS_ITEM_DETAILS="Details"
COM_MENUS_ITEM_FIELD_ALIAS_DESC="The alias is used in the URL when SEF is on."
COM_MENUS_ITEM_FIELD_ALIAS_MENU_DESC="Menu Item to link to."
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.

@mbabker mbabker added this to the Joomla 3.8.4 milestone Dec 23, 2017
@mbabker mbabker merged commit 458d115 into joomla:staging Dec 23, 2017
@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.

None yet

5 participants