? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
8 Nov 2014

batch modals do not display the span6 columns i.e. display the parameters as a single column.

before patch:
wrong batch

after patch
screen shot 2014-11-08 at 11 21 56

this has to be corrected before testing the bunch of PRs to display the parameters consistently

avatar infograf768 infograf768 - open - 8 Nov 2014
avatar jissues-bot jissues-bot - change - 8 Nov 2014
Labels Added: ?
avatar waader
waader - comment - 8 Nov 2014

@test works ltr but not rtl (tested with arabic)
j33_batch_rtl

avatar infograf768
infograf768 - comment - 8 Nov 2014

It does work here in rtl. You may need to purge cache (Using Persian here)
screen shot 2014-11-08 at 15 19 42

avatar infograf768
infograf768 - comment - 8 Nov 2014

I just also corrected another issue in rtl
We had

.row-fluid [class*="span"] {
        margin-left: 15px;
    }
``
instead of margin-right`
avatar waader
waader - comment - 8 Nov 2014

The modal box has the exact appearance as your screenshot. What I meant was, when I am resizing the browser window and the two-columns changes to one column then the labels and fields are not totally right justified as you can see in my screenshot. The are though in ltr. Just a very minor thing.

avatar infograf768
infograf768 - comment - 8 Nov 2014

Corrected the rtl alignment too. Please test

avatar pe7er
pe7er - comment - 8 Nov 2014

I did not see the behavior when I created my patch. Just re-tested with in other resolution and I was able to reproduce the misalignment of the options when my browser is in 800x600px.

avatar infograf768
infograf768 - comment - 8 Nov 2014

Please test this PR

avatar pe7er
pe7er - comment - 8 Nov 2014

@test on default English sample Test data is successful. Thanks JM!

avatar waader
waader - comment - 8 Nov 2014

Perfect! Thank you!

avatar infograf768 infograf768 - close - 8 Nov 2014
avatar infograf768 infograf768 - change - 8 Nov 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-08 15:20:38
avatar infograf768
infograf768 - comment - 8 Nov 2014

Thanks for testing

Add a Comment

Login with GitHub to post a comment