? ? Pending

User tests: Successful: Unsuccessful:

avatar nasirkhan
nasirkhan
15 Oct 2022

Pull Request for Issue # .

Summary of Changes

Added class previous to the 'Previous' button. Right now there is no additional class available for the 'Previous' button.

Testing Instructions

There will be a new class in the 'Previous' button.

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

One new class was added to the button. Functionally and visually there will be no changes. By using this new class, it will be be easier to write button specific styles easily.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar nasirkhan nasirkhan - open - 15 Oct 2022
avatar nasirkhan nasirkhan - change - 15 Oct 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Oct 2022
Category Front End Plugins
avatar richard67
richard67 - comment - 15 Oct 2022

I think this should be rebased to 4.3-dev because it’s a new feature or improvement and not a bug fix. So it should go into 4.3.

avatar nasirkhan
nasirkhan - comment - 15 Oct 2022

I think this should be rebased to 4.3-dev because it’s a new feature or improvement and not a bug fix. So it should go into 4.3.

Thanks @richard67. I updated the base branch to 4.3-dev.

The Labels needed to be updated by someone.

avatar brianteeman
brianteeman - comment - 15 Oct 2022

it is a bug. we have a next class so there should have been a previous class

avatar richard67
richard67 - comment - 15 Oct 2022

it is a bug. we have a next class so there should have been a previous class

@nasirkhan Sorry, I was not aware of that. Could you change back to 4.2-dev?

avatar nasirkhan
nasirkhan - comment - 15 Oct 2022

moved back to 4.2-dev

avatar richard67
richard67 - comment - 15 Oct 2022

I have tested this item successfully on 9bee453

I can confirm the issue that the "Next" button has a "next" class, but the "Previous" button has no "previous" class, and that this PR fixes this issue by adding the missing class to the "Previous" button, and finally that this doesn't have a visual impact.

I've tested with the "Your Modules" article from Blog sample data and the standard Cassiopeia template.


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

avatar richard67 richard67 - test_item - 15 Oct 2022 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 15 Oct 2022

I have tested this item successfully on 9bee453


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

avatar ChristineWk ChristineWk - test_item - 15 Oct 2022 - Tested successfully
avatar richard67 richard67 - change - 15 Oct 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 15 Oct 2022

RTC


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

avatar Quy Quy - change - 15 Oct 2022
Labels Added: ? ?
avatar wilsonge
wilsonge - comment - 18 Oct 2022

Thankyou for your first contribution!

avatar wilsonge wilsonge - change - 18 Oct 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-10-18 13:30:07
Closed_By wilsonge
avatar wilsonge wilsonge - close - 18 Oct 2022
avatar wilsonge wilsonge - merge - 18 Oct 2022

Add a Comment

Login with GitHub to post a comment