WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#46416 closed defect (bug) (duplicate)

Moving up the link next to the button when the button is focused/unfocused

Reported by: man4toman's profile man4toman Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.1
Component: Administration Keywords: has-patch close
Focuses: administration Cc:

Description

After installing the plugin from Add Plugins -> Upload Plugin, the link next to the button goes up and down when the button is focused/unfocused.

Please see the attachment.

Attachments (6)

46416.patch​ (443 bytes) - added by man4toman 6 years ago.
The patch
01-default.png​ (29.3 KB) - added by man4toman 6 years ago.
Default
02-focus.png​ (29.3 KB) - added by man4toman 6 years ago.
When focusing
test.gif​ (4.1 MB) - added by man4toman 6 years ago.
Test result
test.mp4​ (2.0 MB) - added by man4toman 6 years ago.
Test result
46416.1.diff​ (459 bytes) - added by ianbelanger 6 years ago.
Updated patch to cover IE and Edge

Change History (19)

@man4toman
6 years ago

The patch

@man4toman
6 years ago

Default

@man4toman
6 years ago

When focusing

#1 @desrosj
6 years ago

  • Keywords reporter-feedback close added

I am unable to reproduce this with a fresh install on WordPress 5.1. I think that a plugin that you have active may be causing this behavior.

Can you please test with no plugins active?

I am going to mark this a candidate for closing because it worksforme with a fresh install.

#2 @man4toman
6 years ago

Hi @desrosj
I'll test it with a fresh install and send the result here.

@man4toman
6 years ago

Test result

#3 @man4toman
6 years ago

@desrosj
I check it with fresh install of WordPress 5.1 without any plugin, but the problem still here.
Please look at the gif file.

@man4toman
6 years ago

Test result

#4 @man4toman
6 years ago

I attach the video of my test, it has better resolution.

#5 @ianbelanger
6 years ago

  • Keywords needs-patch added; has-patch needs-testing reporter-feedback close removed

This issue appears to be a Windows bug only. I tested it in Firefox, Chrome and IE, all of which display the bug, on focus. The proposed patch does fix the bug in FF and Chrome, but does not seem to fix it in IE. I will test some other solutions in IE so we can patch this.

Thanks for reporting @man4toman

#6 @man4toman
6 years ago

Your welcome @ianbelanger
I test my patch again, it seems it doesn't work on Microsoft Edge too.

@ianbelanger
6 years ago

Updated patch to cover IE and Edge

#7 @ianbelanger
6 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

Patch 46416.1.diff changes vertical-align: initial; to vertical-align: inherit; in order to cover the bug in IE and Edge

#8 @man4toman
6 years ago

@ianbelanger
Works fine on FF and chrome too.

#9 @ianbelanger
6 years ago

  • Keywords needs-testing removed

Hi @man4toman, thanks for clarifying. Yes I should've mentioned that this solution should cover all supported browsers for Windows machines

#10 @desrosj
6 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to Future Release

Moving this to Future Release. I do not have a Windows machine. A committer with access to Windows should give this some final testing and review.

Marking commit based on the fact that the solution works and needs a final review.

#11 @ianbelanger
6 years ago

  • Milestone Future Release deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #44836.

#12 @ianbelanger
6 years ago

  • Keywords close added; commit removed

I found that this a duplicate of an older ticket. The discussion should be continued there. Although I do believe that the solution we have on this ticket is better than what has been suggested on #44836. I am going to submit an updated patch on that ticket.

#13 @man4toman
6 years ago

Well done @ianbelanger

Note: See TracTickets for help on using tickets.