? ? Pending

User tests: Successful: Unsuccessful:

avatar hans2103
hans2103
9 May 2021

Pull Request for Issue # .

Summary of Changes

PR #33654 fixed the placement of Joomla button on the first row.
Caused an unwanted styling bug on com_foo
PR #33689 fixed the styling bug on com_foo
Caused the fix from the first PR

This PR also moves some php blocks to equalize the structure of error_login.php, error_full.php and login.php

Testing Instructions

Actual result BEFORE applying this Pull Request

administrator/index.php?option=com_foo

Schermafbeelding 2021-05-09 om 22 29 55

administrator/index.php?option=com_banners&task=tracks.display&format=raw

Schermafbeelding 2021-05-09 om 21 57 28

Expected result AFTER applying this Pull Request

administrator/index.php?option=com_foo

Schermafbeelding 2021-05-09 om 22 29 55

administrator/index.php?option=com_banners&task=tracks.display&format=raw

Schermafbeelding 2021-05-09 om 22 06 45

Documentation Changes Required

avatar hans2103 hans2103 - open - 9 May 2021
avatar hans2103 hans2103 - change - 9 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 May 2021
Category Administration Templates (admin)
avatar sandramay0905
sandramay0905 - comment - 10 May 2021

I can't find a difference in the screenshots of administrator/index.php?option=com_foo before and after pull request.

avatar hans2103
hans2103 - comment - 10 May 2021

@sandramay0905

I can't find a difference in the screenshots of administrator/index.php?option=com_foo before and after pull request.

Because there is no difference in sight... just in HTML.

avatar sandewt
sandewt - comment - 10 May 2021

Actual result BEFORE applying this Pull Request
Note the black text ! Ignore ?

issue-33917

avatar richard67
richard67 - comment - 10 May 2021

@sandewt The black text should have been fixed with PR #33654 2 days ago. Maybe your testing environment hasn't been updated to that yet?

avatar sandewt
sandewt - comment - 10 May 2021

@sandewt The black text should have been fixed with PR #33654 2 days ago.

@richard67 turns out to be true indeed.

avatar sandewt sandewt - test_item - 10 May 2021 - Tested successfully
avatar sandewt
sandewt - comment - 10 May 2021

I have tested this item successfully on 723eb94


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

avatar drmenzelit drmenzelit - test_item - 10 May 2021 - Tested successfully
avatar drmenzelit
drmenzelit - comment - 10 May 2021

I have tested this item successfully on 723eb94


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

avatar drmenzelit drmenzelit - change - 10 May 2021
Status Pending Ready to Commit
avatar drmenzelit
drmenzelit - comment - 10 May 2021

RTC


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

avatar chmst chmst - change - 11 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-11 07:25:11
Closed_By chmst
Labels Added: ? ?
avatar chmst chmst - close - 11 May 2021
avatar chmst chmst - merge - 11 May 2021
avatar chmst
chmst - comment - 11 May 2021

Thanks

Add a Comment

Login with GitHub to post a comment