? Pending

User tests: Successful: Unsuccessful:

avatar YatharthVyas
YatharthVyas
6 May 2021

Pull Request for Issue #33566

Summary of Changes

This PR adds padding to the header to ensure that the cog icon which has been styled with position: absolute does not overlay on top of the header text. If a custom header_class param is set then this will not override its padding.

Testing Instructions

  1. Visit Backend
  2. Refer to the gif below: Shrink your screen width to reach a breakpoint where the Module Header text is long enough to cover the full width of the card OR you can use inspect element to rewrite the header text such that it covers full width or the card.

Before the PR, the cog would overlay on top of the text because of its absolute position and higher z-index.
After the PR, the padding creates a gap that separates the cog icon from the text by wrapping the text to the next line.

Actual result BEFORE applying this Pull Request

padding_before

Expected result AFTER applying this Pull Request

padding_after

Documentation Changes Required

None

avatar YatharthVyas YatharthVyas - open - 6 May 2021
avatar YatharthVyas YatharthVyas - change - 6 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 May 2021
Category Administration Templates (admin)
avatar sandramay0905 sandramay0905 - test_item - 6 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 6 May 2021

I have tested this item successfully on 0ac8696

@YatharthVyas thanks for solving the issue.


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

avatar YatharthVyas
YatharthVyas - comment - 6 May 2021

Closing this PR as the issue is fixed with the Admin repaint update

avatar YatharthVyas YatharthVyas - close - 6 May 2021
avatar YatharthVyas YatharthVyas - change - 6 May 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-05-06 10:08:22
Closed_By YatharthVyas
Labels Added: ?
avatar YatharthVyas YatharthVyas - change - 6 May 2021
Title
[4.0] Backend - Add padding to module card headers
[4.0] (Issue Resolved) Backend - Add padding to module card headers
avatar YatharthVyas YatharthVyas - edited - 6 May 2021

Add a Comment

Login with GitHub to post a comment