Skip to:
Content

BuddyPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#5439 closed enhancement (fixed)

remove xprofile visibility controls animation

Reported by: DJPaul Owned by: djpaul
Milestone: 2.0 Priority: normal
Severity: normal Version:
Component: Core Keywords:
Cc:

Description

The xprofile "change" visibility controls, whether front-end or in wp-admin, use a fairly common jQuery fade/slide animation. This feels pretty Web 2.0 and dated to me; removing the animation makes the UI feel more responsive IMO.

Current:
http://f.cl.ly/items/1b3a3q2t3A3h2V0J202y/with-animation.gif

Proposed:
http://f.cl.ly/items/1G2u1C1A2H0n3N3x3K3V/without-animation.gif

If we want to do this, I'm happy to work on the change.

Change History (6)

#1 @hnla
7 years ago

Gee thanks Paul I added those fades amongst other things, nice to feel dated! :( #4224

Personally think they look fine, but change by all means it's just a matter of removing the slidedown & fadeup properties in js. If you want to get funky explore using CSS 'Transitions' for cool alternative effects; nice gifs though :)

#2 follow-up: @boonebgorges
7 years ago

This seems fine to me, but how does it interface with #5352? Are these inline settings being removed?

#3 in reply to: ↑ 2 @DJPaul
7 years ago

Replying to boonebgorges:

This seems fine to me, but how does it interface with #5352? Are these inline settings being removed?

That's a good question we ought to have in that other thread; I don't know what plans John has for it.

#4 @DJPaul
7 years ago

  • Milestone changed from Awaiting Review to 2.0

#5 @djpaul
7 years ago

  • Owner set to djpaul
  • Resolution set to fixed
  • Status changed from new to closed

In 8021:

xProfile: change the field visibility JS to show/hide the controls immediately, rather than with a slide/fade transition.

This helps improve the perceived performance of the visibility
controls. The code quality of these parts of the JS has also been
improved.

Fixes #5439

#6 @DJPaul
5 years ago

  • Component changed from General - UX/UI to Core
Note: See TracTickets for help on using tickets.