Problem/Motivation

The active horizontal tab has different padding applied causing misalignment

Active tab:

Other tab:

Steps to reproduce

Use horizontal field group + gin theme

Proposed resolution

Make the tabs have consistent padding

Remaining tasks

Merge request

User interface changes

Horizontal active tab and non-active tabs will align

API changes

N/A

Data model changes

N/A

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

scott_euser created an issue. See original summary.

scott_euser’s picture

Issue summary: View changes
StatusFileSize
new19.44 KB
new12.45 KB

scott_euser’s picture

Status: Active » Needs review
StatusFileSize
new13.93 KB

After fix:

pilot3’s picture

I want to review this, but I don't see the same problem on my site. I use Gin as admin theme and had field group module set horizontal.

scott_euser’s picture

It's within a Paragraph type in my example, perhaps that causes the issue

scott_euser’s picture

I can check and compare tomorrow and get back to you

scott_euser’s picture

Status: Needs review » Closed (outdated)

Hmmm can't reproduce any more. Must have been fixed elsewhere!

scott_euser’s picture

Status: Closed (outdated) » Needs review
StatusFileSize
new1.16 MB

Actually yep this still exists:

Screencast toggling on and off the padding on tab

kiseleva.t’s picture

StatusFileSize
new448 bytes

Exported MR into patch to use in composer.

grevil’s picture

Version: 8.x-3.x-dev » 4.x-dev
grevil’s picture

Status: Needs review » Postponed (maintainer needs more info)

Is this still reproducible in latest 4.x dev?

scott_euser’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new12.67 KB

Screenshot of the tabs still being misaligned

Yep still an issue in 4.x unfortunately. MR still solves.

karengrey’s picture

Status: Needs review » Reviewed & tested by the community

Can confirm CSS change fixes the issue
tested same scenario using paragraphs.

anybody’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for the fixes here @all

  • anybody committed f163018d on 4.x authored by scott_euser
    Issue #3462300 by scott_euser, kiseleva.t, pilot3, grevil, karengrey:...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.