J4 Issue ? Failure

User tests: Successful: Unsuccessful:

avatar hardik-codes
hardik-codes
4 Mar 2019

Pull Request for Issue #23815 .

Summary of Changes

Changes in administrator/templates/atum/index.php

Testing Instructions

Open the articles and reduce the browser viewport size

Expected result

Toolbar displayed/hidden when toggle button is clicked

Actual result

The toolbar toggle button does nothing

Documentation Changes Required

None

avatar hardik-codes hardik-codes - open - 4 Mar 2019
avatar hardik-codes hardik-codes - change - 4 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Mar 2019
Category Administration Templates (admin)
avatar hardik-codes
hardik-codes - comment - 4 Mar 2019

@ciar4n please have a look and suggest me changes if any.

avatar hardik-codes hardik-codes - change - 4 Mar 2019
Title
small screen toolbar toggle fix
[4.0]small screen toolbar toggle fix
avatar hardik-codes hardik-codes - edited - 4 Mar 2019
7c1896e 4 Mar 2019 avatar hardik-codes fix
avatar hardik-codes hardik-codes - change - 4 Mar 2019
Labels Added: ?
avatar hardik-codes
hardik-codes - comment - 5 Mar 2019

@ciar4n please review the changes

avatar hardik-codes
hardik-codes - comment - 8 Mar 2019

Members please test this PR

avatar flyingwombats
flyingwombats - comment - 13 Mar 2019

Sorry, I am not sure what we should see. Are you talking about the Toggle menu button when in
/administrator/index.php?option=com_fields&context=com_content.article
The toggle seems to work before the patch is applied even when the page is made small
Could you show an image of it in the broken and patched state so I can work out if I am looking at the right thing.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24083.
avatar hardik-codes
hardik-codes - comment - 13 Mar 2019

@flyingwombats
Resize your browser window and you will find a toolbar button
tp-1

Earlier this button was not toggleable but after the patch it is toggleable.

avatar Bodge-IT
Bodge-IT - comment - 14 Mar 2019

Tested this and the toggle now works but no styling and no cursor change on hover, were these excpected?


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

avatar hardik-codes
hardik-codes - comment - 14 Mar 2019

@Bodge-IT
I think the only intention was to make the toolbar button toggleable.

avatar Bodge-IT
Bodge-IT - comment - 14 Mar 2019

I have tested this item successfully on 0ebeb6b

Toggle button toggles after patch.

Note: No cursor indication of "button"


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

avatar Bodge-IT Bodge-IT - test_item - 14 Mar 2019 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Mar 2019

Note this fails almost every measure of accessibility

avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Apr 2019
Title
[4.0]small screen toolbar toggle fix
[4.0] small screen toolbar toggle fix
avatar franz-wohlkoenig franz-wohlkoenig - edited - 19 Apr 2019
avatar Quy Quy - change - 18 Jul 2019
Closed_Date 2019-07-18 17:26:36 2019-07-18 17:26:37
avatar Quy Quy - close - 18 Jul 2019
avatar Quy Quy - change - 18 Jul 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-07-18 17:26:36
Closed_By Quy
avatar Quy
Quy - comment - 18 Jul 2019

This is no longer an issue with the new backend template. Please test PR #25570.

toolbar


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

Add a Comment

Login with GitHub to post a comment