? Failure
Related to # 4754
Referenced as Related to: # 4754

User tests: Successful: Unsuccessful:

avatar garima-agal
garima-agal
17 Oct 2014

. We can apply validation, to make selection first

1
2

avatar garima-agal garima-agal - open - 17 Oct 2014
avatar jissues-bot jissues-bot - change - 17 Oct 2014
Labels Added: ?
avatar losedk losedk - test_item - 17 Oct 2014 - Tested successfully
avatar losedk
losedk - comment - 17 Oct 2014

Works fine. Perhaps we should take it a step further at hide the batch button until something is selected?

avatar brianteeman brianteeman - change - 17 Oct 2014
Category UI/UX
avatar brianteeman brianteeman - change - 17 Oct 2014
The description was changed
Title
Batch process pop-up gets open even without selection. We can apply vali...
Batch process pop-up gets open even without selection
Rel_Number 4754
Relation Type Related to
avatar allanlhansen allanlhansen - test_item - 17 Oct 2014 - Tested successfully
avatar brianteeman
brianteeman - comment - 17 Oct 2014

@losedk the UX sprint last year tried to get that sort of change in and it was rejected. :(

On the basis of two good tests I am setting this RTC

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

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

@brianteeman ahh okay :S

avatar zero-24
zero-24 - comment - 17 Oct 2014

@garima-agal It looks like we have a failed Travis test:


There was 1 failure:
1) JToolbarButtonHelpTest::testFetchButton
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'<button onclick="Joomla.popupWindow('help/en-GB/JHELP_CONTENT_ARTICLE_MANAGER.html', 'JHELP', 700, 500, 1)" rel="help" class="btn btn-small">
+'<button onclick="Joomla.popupWindow('help/en-GB/Content_Article_Manager.html', 'Help', 700, 500, 1)" rel="help" class="btn btn-small">
    <span class="icon-question-sign"></span>
-   JTOOLBAR_HELP</button>
+   Help</button>
 '
/home/travis/build/joomla/joomla-cms/tests/unit/suites/libraries/cms/toolbar/button/JToolbarButtonHelpTest.php:102

See: https://travis-ci.org/joomla/joomla-cms/jobs/38266232

Can you have a look into it?

avatar vijaykhollam vijaykhollam - test_item - 18 Oct 2014 - Tested successfully
avatar sandeepghule sandeepghule - test_item - 18 Oct 2014 - Tested successfully
avatar phproberto phproberto - change - 23 Oct 2014
Status Ready to Commit Needs Review
avatar Bakual
Bakual - comment - 8 Nov 2014

Merged, thanks.
After some tests in my fork I came to the conclusion that the failed tests must be due to some outdated branch and not related to the PR.

avatar Bakual Bakual - close - 8 Nov 2014
avatar Bakual Bakual - change - 8 Nov 2014
Status Needs Review Closed
Closed_Date 0000-00-00 00:00:00 2014-11-08 16:46:28
avatar infograf768
infograf768 - comment - 25 Dec 2014

FYI, just corrected this
de6b74a

Add a Comment

Login with GitHub to post a comment