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

Select or Create in Menu #14637

Merged
merged 4 commits into from
Mar 15, 2017
Merged

Select or Create in Menu #14637

merged 4 commits into from
Mar 15, 2017

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Mar 15, 2017

Summary of Changes

We now have the great ability to create an article, contact, category directly at the point you create a menu item. This PR updates the tooltip displayed to reflect that you can Select AND create. I also took the opportunity to standardise the strings across the component for consistency

Testing Instructions

On review or by checking the tooltips when creating menu items
screenshotr12-23-52

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Mar 15, 2017
@alikon
Copy link
Contributor

alikon commented Mar 15, 2017

I have tested this item ✅ successfully on 43cf3cb


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

1 similar comment
@jeckodevelopment
Copy link
Member

I have tested this item ✅ successfully on 43cf3cb


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

@jeckodevelopment
Copy link
Member

RTC


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

@jeckodevelopment jeckodevelopment removed Language Change This is for Translators PR-staging labels Mar 15, 2017
@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 15, 2017
@jeckodevelopment jeckodevelopment added the Language Change This is for Translators label Mar 15, 2017
@@ -305,7 +305,7 @@ JGLOBAL_CATEGORY_ORDER_DESC="The order that categories will show in."
JGLOBAL_CATEGORY_ORDER_LABEL="Category Order"
JGLOBAL_CENTER="Center"
JGLOBAL_CHECK_ALL="Check All"
JGLOBAL_CHOOSE_CATEGORY_DESC="Choose a category from the list."
JGLOBAL_CHOOSE_CATEGORY_DESC="Select or create a category to be displayed."
Copy link
Contributor

Choose a reason for hiding this comment

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

OK Two things here:

  1. You have a double space beween "select" and "or"
  2. Given this is in the global file and people may not have the create category option enabled do we feel comfortable that this isn't against b/c?

@ghost
Copy link

ghost commented Mar 15, 2017

this PR have no Milestone.

@jeckodevelopment jeckodevelopment added this to the Joomla 3.7.0 milestone Mar 15, 2017
@rdeutz rdeutz merged commit 7472468 into joomla:staging Mar 15, 2017
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 15, 2017
@brianteeman brianteeman deleted the menu-select branch March 15, 2017 20:26
Bakual pushed a commit that referenced this pull request Mar 16, 2017
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

6 participants