WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#48490 closed enhancement (fixed)

Login failure "shake" should use CSS animation

Reported by: markjaquith Owned by: markjaquith
Priority: normal Milestone: 5.4
Component: Login and Registration Version:
Severity: trivial Keywords: has-patch
Cc: Focuses: ui, administration

Description (last modified by markjaquith)

The login failure "shake" uses really terrible old JavaScript to simulate the shake by moving the box. It's "choppy" by design. We have much better options now, like CSS animations.

Attachments (2)

48490.diff​ (2.0 KB ) - added by markjaquith 7 years ago.
48490.1.diff​ (1.9 KB ) - added by valentinbora 7 years ago.
Fix indentation upon 48490.diff​ in src/wp-login.php

Download all attachments as: .zip

Change History (8)

@markjaquith
7 years ago

#1 @markjaquith
7 years ago

48490.diff is a much simpler and faster implementation, using translateX()

#2 @markjaquith
7 years ago

  • Description modified (diff)
  • Keywords has-patch added
  • Milestone Awaiting Review → 5.4

@valentinbora
7 years ago

Fix indentation upon 48490.diff​ in src/wp-login.php

#3 @valentinbora
7 years ago

Confirmed @markjaquith's patch functionality, good job.

This ticket was mentioned in ​Slack in #core by david.baumwald. ​View the logs.
7 years ago

#5 @SergeyBiryukov
7 years ago

  • Component Administration → Login and Registration

#6 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status assigned → closed

In 47226:

Login and Registration: Use CSS animation instead of old JavaScript code for "shake" animation on login failure.

Props markjaquith, valentinbora.
Fixes #48490.

Note: See TracTickets for help on using tickets.