? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
26 Aug 2022

Pull request for #38512

When there is only a next button it is no longer on the right as intended. this was due to a change in the markup.

As this a change in the scss you will need to either npm run build:css or use one of the prebuilt packages

Before

image

After

image

avatar brianteeman brianteeman - open - 26 Aug 2022
avatar brianteeman brianteeman - change - 26 Aug 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2022
Category Repository NPM Change Front End Plugins
avatar brianteeman brianteeman - change - 26 Aug 2022
The description was changed
avatar brianteeman brianteeman - edited - 26 Aug 2022
avatar sandewt
sandewt - comment - 27 Aug 2022

I'm not a designer, but the button looks very small, please make it a bit bigger. Add p-2.

E.g. class="btn p-2 btn-sm btn-secondary next"


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

avatar brianteeman
brianteeman - comment - 27 Aug 2022

I am not changing the size, just fixing the bug

avatar sandewt
sandewt - comment - 27 Aug 2022

I am not changing the size, just fixing the bug

That would be a new PR.

avatar sandewt
sandewt - comment - 27 Aug 2022

English: successful test

issue_38612

Arabic: successful test ???
Can anyone confirm that this test is also successful.

issue_38612-2

avatar N6REJ N6REJ - test_item - 28 Aug 2022 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 28 Aug 2022

@N6REJ you have a template override in place

avatar N6REJ
N6REJ - comment - 28 Aug 2022

@N6REJ you have a template override in place

nope, I hadn't ran npm ci

avatar N6REJ
N6REJ - comment - 28 Aug 2022

I have tested this item successfully on 872838f

works after running npm ci as instructed


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

avatar N6REJ N6REJ - test_item - 28 Aug 2022 - Tested successfully
avatar N6REJ N6REJ - change - 28 Aug 2022
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 28 Aug 2022

@N6REJ you have a template override in place

nope, I hadn't ran npm ci

well you have an extra class there thats not in core docs-creator

avatar N6REJ
N6REJ - comment - 28 Aug 2022

@N6REJ you have a template override in place

nope, I hadn't ran npm ci

well you have an extra class there thats not in core docs-creator

that is fresh 4.2 install with blog sample data . I'll check the *ampp stack

avatar N6REJ
N6REJ - comment - 28 Aug 2022

@brianteeman ok, it comes from an extension for chrome called Docs Creator

avatar brianteeman
brianteeman - comment - 28 Aug 2022

see I'm not daft

avatar richard67
richard67 - comment - 28 Aug 2022

@brianteeman Is the screenshot for RTL in @sandewt 's comment above ok? He was not sure if his test with Arabic can be counted as success. I think it's because of the position of the button, it seems to have some margin left which it doesn's have with LTR.

@N6REJ As there was one test only, i.e. yours, it cannot be RTC. @sandewt test for RTL (Arabic) was not clear.

avatar richard67 richard67 - change - 28 Aug 2022
Status Ready to Commit Pending
Labels Added: NPM Resource Changed ?
avatar richard67
richard67 - comment - 28 Aug 2022

Back to pending.


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

avatar brianteeman
brianteeman - comment - 28 Aug 2022

Yes that's fine as an RTL trst

avatar richard67 richard67 - alter_testresult - 28 Aug 2022 - sandewt: Tested successfully
avatar richard67
richard67 - comment - 28 Aug 2022

@sandewt As @brianteeman confirmed that your test for RTL direction with Arabic is ok, I was so free to set your test result to success.

avatar richard67 richard67 - change - 28 Aug 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 28 Aug 2022

RTC


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

avatar sandewt
sandewt - comment - 28 Aug 2022

I was so free to set your test result to success.

@richard67 that's OK. Thks.


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

avatar wilsonge wilsonge - change - 6 Sep 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-09-06 10:03:12
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 6 Sep 2022
avatar wilsonge wilsonge - merge - 6 Sep 2022
avatar wilsonge
wilsonge - comment - 6 Sep 2022

Thanks!

avatar brianteeman
brianteeman - comment - 6 Sep 2022

thanks

Add a Comment

Login with GitHub to post a comment