? ? Success

User tests: Successful: Unsuccessful:

avatar Flow87
Flow87
9 Oct 2015

Provide possible solution for issue #5570

Request:
Placeholder message for tag field should be "Type or select some options".

More information:

The input field of tags which you find at the article manager in the back end has the placeholder message "Select an option". This can lead the user to think that there it is only the possibility to select one of the already given tags. But the actual options you have through selecting the input field are:

  1. Select one tag from available list

  2. Select multiple tags from available list

  3. Type a tag

  4. Type multiple tags

  5. Combination of selected and typed tags

In order to make the bahavior of this input field clearer the placeholder message should be changed to something like: "Type or select some options"

The solution for this issue:

  1. The string used as placeholder is from the global variable "JGLOBAL_SELECT_SOME_OPTIONS" in the "en-GB.ini".

  2. In the "formbehavior.php" this variable is used as placeholder when the field is marked as "multiple" in a method named "chosen".

  3. Created a new global variable named "JGLOABL_TYPE_OR_SELECT_SOME_OPTIONS" in "en-GB.ini" which contains the string "Type or select some options".

  4. Changed the method "chosen" in "formbehavior.php" so if the field is marked as "multiple" the string given for the placeholder message is the created "JGLOABL_TYPE_OR_SELECT_SOME_OPTIONS".

How to test:

-Log into the administrator panel
-Go to article manager
-Create new article
-Check placeholder message

Expected result after solution:
The placeholder message of the tag field will now be "Type or select some options."

Worked as a group on that issue: @icampus, @FPerisa, @kathastaden and @xsability

avatar Flow87 Flow87 - open - 9 Oct 2015
avatar Flow87 Flow87 - change - 9 Oct 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Oct 2015
Labels Added: ? ?
avatar zero-24 zero-24 - change - 10 Oct 2015
Category Administration Language & Strings UI/UX
avatar zero-24 zero-24 - change - 10 Oct 2015
Easy No Yes
avatar zero-24 zero-24 - alter_testresult - 10 Oct 2015 - wschouwe: Tested successfully
avatar zero-24
zero-24 - comment - 10 Oct 2015

successful tested by @wschouwe here: 3eb605c one more tester ;)


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

avatar brianteeman
brianteeman - comment - 10 Oct 2015

@wschouwe How can that be a successful test. That was the original string not the changed string

avatar crommie crommie - test_item - 10 Oct 2015 - Tested successfully
avatar crommie
crommie - comment - 10 Oct 2015

I have tested this item :white_check_mark: successfully on d8f1b7e

Tested according to instructions. Works. Nice.


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

avatar zero-24 zero-24 - change - 10 Oct 2015
Milestone
avatar zero-24 zero-24 - change - 10 Oct 2015
Status Pending Ready to Commit
avatar roland-d roland-d - change - 22 Oct 2015
Status Ready to Commit Pending
Labels
avatar joomla-cms-bot joomla-cms-bot - change - 22 Oct 2015
Milestone Removed:
avatar roland-d
roland-d - comment - 22 Oct 2015

Removed RTC status because of Brian's remark that the first test can't be successful. Please make another test on this.


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

avatar infograf768
infograf768 - comment - 23 Oct 2015

The new string should also be added in frontend en-GB.ini as it is used there.

screen shot 2015-10-23 at 08 41 20

avatar si-wi si-wi - test_item - 24 Oct 2015 - Tested successfully
avatar si-wi
si-wi - comment - 24 Oct 2015

I have tested this item :white_check_mark: successfully on d8f1b7e

tested successfully according "How to test"
Button label before patch: "Select some options"
Button label after patch: "Type or select some options"


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

avatar zero-24
zero-24 - comment - 24 Oct 2015

@Flow87 can you implement the new string also to the frontend en-GB.ini see: https://github.com/Flow87/joomla-cms/blob/staging/language/en-GB/en-GB.ini

avatar wojsmol
wojsmol - comment - 26 Oct 2015
avatar NLRoosj NLRoosj - test_item - 12 Dec 2015 - Tested successfully
avatar NLRoosj
NLRoosj - comment - 12 Dec 2015

I have tested this item :white_check_mark: successfully on d8f1b7e


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

avatar brianteeman
brianteeman - comment - 12 Dec 2015

This cant be merged #8041 (comment)

avatar wojsmol
wojsmol - comment - 13 Dec 2015
avatar brianteeman
brianteeman - comment - 10 Mar 2016

This has been waiting for 3 months just to have a new string added to the PR - not worth waiting any longer. I have remade the PR with the missing string at #9360


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

avatar brianteeman brianteeman - change - 10 Mar 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-03-10 13:00:43
Closed_By brianteeman
Labels
avatar brianteeman brianteeman - change - 10 Mar 2016
Labels

Add a Comment

Login with GitHub to post a comment