Make WordPress Core

Opened 7 years ago

Closed 4 years ago

#35166 closed defect (bug) (fixed)

strange effect in tab active

Reported by: hrohh's profile Hrohh Owned by: misfist's profile misfist
Milestone: 5.2 Priority: normal
Severity: normal Version: 4.4
Component: Administration Keywords: good-first-bug has-patch close reporter-feedback
Focuses: ui Cc:

Description

If I click on tab, there is strange effect on bottom-border. I figure out, that problem is global

a { 
    -webkit-transition-property: border,background,color;
    transition-property: border,background,color;
}


In admin css there should be:

/* fix for strange effect */
.nav-tab-active {
    -webkit-transition-property: background,color;
    transition-property: background,color;
}

Attachments (11)

35166-about-page.gif (32.1 KB) - added by ocean90 7 years ago.
35166-menus.gif (20.8 KB) - added by ocean90 7 years ago.
35166.diff (473 bytes) - added by coderste 7 years ago.
about-chrome.png (67.4 KB) - added by misfist 6 years ago.
About Page Chrome
about-ff.png (65.7 KB) - added by misfist 6 years ago.
About Page Firefox
nav-menus-chrome.png (53.1 KB) - added by misfist 6 years ago.
Nav Menus Chrome
nav-menus-ff.png (62.6 KB) - added by misfist 6 years ago.
Nav Menus Firefox
35166.2.diff (791 bytes) - added by rleeson 6 years ago.
Border scaling workarounds
35166.patch (456 bytes) - added by riddhiehta02 6 years ago.
Patch modify and added border to the element on hover effect.
9fbc95f115e4597a1a1bf8d7eb60f11e.gif (262.9 KB) - added by audrasjb 4 years ago.
294e68153d79aed466ffdeca5767883f.gif (238.1 KB) - added by audrasjb 4 years ago.

Download all attachments as: .zip

Change History (29)

#1 @afercia
7 years ago

  • Component changed from General to Administration

Hi @Hrohh and thanks for your report. As far as I can tell, Core doesn't use Tabs animated with JavaScript which will expose that CSS transition. In the "About" screens, "Menus" screen, and Network "Sites" screens, each tab reloads a new page.

Sometimes Plugins use JavaScript based Tabs with the Core styles, in this case the bottom border will be transitioned exactly like the background color. I wouldn't consider this a bug. Curious in which plugin have you noticed this? Would like to double check what's happening :)

#2 @afercia
7 years ago

  • Keywords reporter-feedback added

@ocean90
7 years ago

#3 @ocean90
7 years ago

  • Keywords needs-patch good-first-bug added; reporter-feedback removed
  • Milestone changed from Awaiting Review to Future Release

@afercia This doesn't seem to be related to JavaScript. I can reproduce it on the about page, see 35166-about-page.gif, and on the menu screen, see 35166-menus.gif.

#4 follow-up: @afercia
7 years ago

@ocean90 interesting, I think that's a different issue that happens in the responsive view, seems just when hovering on the active tab, right? By the way @Hrohh mentioned a strange effect when clicking on a tab:

If I click on tab ...

and since when clicking on the tabs in the admin there's a page reload, was wondering what the strange effect is. Some plugins use tabs for "in page" content instead and maybe the animation on the border is causing some visual glitch. Couldn't reproduce though.

#5 in reply to: ↑ 4 @bhargavbhandari90
7 years ago

Replying to afercia: Yes what effect I don't understand when clicking on that tab.

@ocean90 interesting, I think that's a different issue that happens in the responsive view, seems just when hovering on the active tab, right? By the way @Hrohh mentioned a strange effect when clicking on a tab:

If I click on tab ...

and since when clicking on the tabs in the admin there's a page reload, was wondering what the strange effect is. Some plugins use tabs for "in page" content instead and maybe the animation on the border is causing some visual glitch. Couldn't reproduce though.

Last edited 7 years ago by bhargavbhandari90 (previous) (diff)

#6 @coderste
7 years ago

Regarding the issue @ocean90 pointed out. I believe this is because when you hover on an item that is active there is no CSS inside the @media screen and (max-width: 600px) for an active tab therefore it is using the desktop CSS which is using a border-bottom: 1px solid #f1f1f1; on hover hence it removing the bottom border.

I have attached a patch that fixes the issue but if it doesn't relate to this ticket I will create a new one :)

@coderste
7 years ago

#7 @trot77
7 years ago

  • Keywords has-patch added; needs-patch removed

#8 @trot77
7 years ago

Patch looks good. Tested it and worked fine.

#9 @fischfood
7 years ago

Confirming fixed on Menus area and about.php

#10 @toddmilliken
7 years ago

Patch looks good on credits.php. Tested on Chrome / Firefox on Mac.

#11 @lukecavanagh
7 years ago

@Hrohh

Patch works well.

@misfist
6 years ago

About Page Chrome

@misfist
6 years ago

About Page Firefox

@misfist
6 years ago

Nav Menus Chrome

@misfist
6 years ago

Nav Menus Firefox

#12 @misfist
6 years ago

My understanding of this issue is that on smaller screens (max-width: 600px) only, the bottom border on the active tab appears to disappear on hover.

I tested the patch in Firefox 50.0.2 and Chrome 54.0.2840.98 (OS X 10.10.5) and it seems to work: on smaller screens, the bottom border remains on hover.

@rleeson
6 years ago

Border scaling workarounds

#13 @rleeson
6 years ago

Chrome scales the border to 80% of the requested size. I attempted transform scale overrides, to reset the scale to 1, which were unsuccessful. Instead, I set the bottom border color to transparent, which should work across any color theme.

Additionally, there is an image report showing another border issue, with the Menu Page and Category sub-selection boxes, due to the same scaling problem. This is slightly different, as it needs to overlap the differently colored border beneath it. I doubled the border size, and continued to have it match the background color.

Patch 35166.2

Last edited 6 years ago by rleeson (previous) (diff)

@riddhiehta02
6 years ago

Patch modify and added border to the element on hover effect.

#15 @DrewAPicture
6 years ago

  • Owner set to misfist
  • Status changed from new to assigned

Assigning ownership to mark the good-first-bug as "claimed".

This ticket was mentioned in Slack in #core by sergey. View the logs.


4 years ago

#17 @audrasjb
4 years ago

  • Keywords close reporter-feedback added

Hi there,

Looks like the issue doesn't exists anymore.
See animated screenshots above.

Adding reporter-feedback and close keyword (before closing as worksforme)

Cheers,
Jb

#18 @SergeyBiryukov
4 years ago

  • Milestone changed from Future Release to 5.2
  • Resolution set to fixed
  • Status changed from assigned to closed

Appears to be fixed in [45276].

Note: See TracTickets for help on using tickets.