? Success
Referenced as Related to: # 4767

User tests: Successful: Unsuccessful:

avatar RCheesley
RCheesley
17 Oct 2014

This fixes issue #4767 by implementing a popup to warn if no items have been selected.

The alert box should be displayed for Mark as Read & Mark as Unread on Messaging page.

avatar RCheesley RCheesley - open - 17 Oct 2014
avatar jissues-bot jissues-bot - change - 17 Oct 2014
Labels Added: ?
avatar nicksavov nicksavov - change - 17 Oct 2014
Labels Added: ?
avatar brianteeman brianteeman - change - 17 Oct 2014
The description was changed
avatar mtmeyrick mtmeyrick - test_item - 17 Oct 2014 - Tested successfully
avatar zero-24 zero-24 - change - 17 Oct 2014
Category Administration UI/UX
avatar anibalsanchez
anibalsanchez - comment - 17 Oct 2014

@test OK, works perfectly!

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

avatar brianteeman brianteeman - test_item - 17 Oct 2014 - Tested successfully
avatar brianteeman brianteeman - alter_testresult - 17 Oct 2014 - anibalsanchez : Not tested
avatar brianteeman brianteeman - alter_testresult - 17 Oct 2014 - anibalsanchez : Tested successfully
avatar brianteeman brianteeman - alter_testresult - 17 Oct 2014 - brianteeman: Not tested
avatar brianteeman
brianteeman - comment - 17 Oct 2014

Great work and congrats on your first Pull Request setting to RTC

avatar brianteeman brianteeman - change - 17 Oct 2014
Status Pending Ready to Commit
avatar JoomliC
JoomliC - comment - 17 Oct 2014

@RCheesley, for Travis to success, you need to add a space after comma : ", true" ;-)

In my opinion, is alert popup box needed, as we have already working the J alert message ?

Before PR - Joomla alert message :
capture d ecran 2014-10-18 a 01 30 57

After PR - Alert popup :
capture d ecran 2014-10-18 a 01 30 06

I know it is like that elsewhere, but some users never see alert popup (a browser could block them), it could be more secure to used Joomla integrated alert message system everywhere ?

In case Alert popup is the recommended usage, @test success !

Welcome @RCheesley for your first PR, and not the last one ? ;-)

avatar RCheesley
RCheesley - comment - 18 Oct 2014

OK I'll take a look at this and see what I can work out myself ;)

avatar jissues-bot jissues-bot - change - 18 Oct 2014
Labels Added: ?
avatar phproberto phproberto - close - 18 Oct 2014
avatar phproberto phproberto - change - 18 Oct 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-10-18 02:56:44

Add a Comment

Login with GitHub to post a comment