Skip to content

Post install messages#8457

Merged
roland-d merged 2 commits intojoomla:stagingfrom
brianteeman:walltext
May 7, 2016
Merged

Post install messages#8457
roland-d merged 2 commits intojoomla:stagingfrom
brianteeman:walltext

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

@brianteeman brianteeman commented Nov 17, 2015

Remove extra text and change Review to Read. It really isnt necessary to have a wall of text explaining that if you click on the button immediately below called read messages then this will allow you ro read the message.

Avoid over-communication

Be explicit wherever necessary but don't explain the obvious. On the web scanning is the norm and too much text weakens the effectiveness of the message - it does not enhance

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Nov 17, 2015
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 17, 2015

I have tested this item ✅ successfully on 30b00b9

Works


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

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 17, 2015

RTC as it is a language change. Let the PLT decide to go with it. Thanks.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 17, 2015
@infograf768
Copy link
Copy Markdown
Member

hmm...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if we change it, it is not crystal clear what the "Reset Messages" button is for, wouldn't "Unread Messages" for the button better. Then it is clear.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No because the word 'unread' has two different meanings.
It would normally mean 'show me the messages that have not been read yet'
and not 'mark the messages that have been read already as to be read'

Using the word unread to mean undo the action of reading is not real
English.
On 20 Nov 2015 9:05 am, "Robert Deutz" notifications@github.com wrote:

In administrator/language/en-GB/en-GB.com_postinstall.ini
#8457 (comment):

@@ -8,7 +8,7 @@ COM_POSTINSTALL_BTN_HIDE="Hide this message"
COM_POSTINSTALL_BTN_RESET="Reset Messages"
COM_POSTINSTALL_CONFIGURATION="Post-installation Messages: Options"
COM_POSTINSTALL_LBL_MESSAGES="Post-installation and Upgrade Messages"
-COM_POSTINSTALL_LBL_NOMESSAGES_DESC="You have already seen and hidden all the messages. To display them again please select the Reset Messages button below."
+COM_POSTINSTALL_LBL_NOMESSAGES_DESC="You have read all the messages."

if we change it, it is not crystal clear what the "Reset Messages" button
is for, wouldn't "Unread Messages" for the button better. Then it is clear.


Reply to this email directly or view it on GitHub
https://github.com/joomla/joomla-cms/pull/8457/files#r45447864.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

but reset seems to me also not the right word, we don't "reset messages" we reset the read/unread state of a message

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You are thinking as a developer not as a speaker of English
On 20 Nov 2015 5:28 pm, "Robert Deutz" notifications@github.com wrote:

In administrator/language/en-GB/en-GB.com_postinstall.ini
#8457 (comment):

@@ -8,7 +8,7 @@ COM_POSTINSTALL_BTN_HIDE="Hide this message"
COM_POSTINSTALL_BTN_RESET="Reset Messages"
COM_POSTINSTALL_CONFIGURATION="Post-installation Messages: Options"
COM_POSTINSTALL_LBL_MESSAGES="Post-installation and Upgrade Messages"
-COM_POSTINSTALL_LBL_NOMESSAGES_DESC="You have already seen and hidden all the messages. To display them again please select the Reset Messages button below."
+COM_POSTINSTALL_LBL_NOMESSAGES_DESC="You have read all the messages."

but reset seems to me also not the right word, we don't "reset messages"
we reset the read/unread state of a message


Reply to this email directly or view it on GitHub
https://github.com/joomla/joomla-cms/pull/8457/files#r45494426.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So then explain me as a developer what "Reset Messages" for a speaker of English means, please.

@pjwiseman
Copy link
Copy Markdown
Contributor

I'd agree with JM. I'm not sure about this change, and would be inclined to reject it s it currently stands. I think the term "Review" is likely to be clearer if English isn't your first language (it avoids the awkward double meanings). I don't mind the extra information on the Post Installation Messages page as it is providing additional value, and is otherwise pretty empty.

I would support a change to remove all the extra information on the cpanel page and just leave the title and the button (without renaming it). Whilst it may have had value when initially introduced, it's now cumbersome.

Given two people questioning the change, I'm moving it back to pending as it needs additional support or changes. PLT don't need to be bothered with it yet, IMO.

(PS. I'm just back for a brief stint in the JBS. I'm not prepared to spend much until I can get all the sites I maintain up to V3. Too many third party extensions won't upgrade without issues.)


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

@pjwiseman
Copy link
Copy Markdown
Contributor

Seems I've lost my ability to make status changes (or maybe never had it - it's been a while).


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

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 25, 2015

done you got it back @pjwiseman let me know if you need help with the new Tracker and contact me e.g. on glip welcome back 😄


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

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 25, 2015
@anibalsanchez
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 30b00b9

It works OK


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

@roland-d roland-d added this to the Joomla 3.6.0 milestone May 7, 2016
@roland-d roland-d merged commit 59b2174 into joomla:staging May 7, 2016
@roland-d
Copy link
Copy Markdown
Contributor

roland-d commented May 7, 2016

Merged, thanks everybody for participating.

@brianteeman brianteeman deleted the walltext branch May 9, 2016 20:28
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.

8 participants