fix flickering of warning box on start page

VERIFIED FIXED in 2011-10-18

Status

VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: atopal, Assigned: rrosario)

Tracking

unspecified
2011-10-18

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
The warning box on the start page shows all content first and then hides it. That is rather confusing and on slow machines/connections people have time to read the full warning before it goes away.

Please insert that box after the page is loaded, and only show the appropriate showfor content instead of hiding it after loading it.
(Assignee)

Comment 1

7 years ago
I don't believe we want to add ajax here. We need to figure out how to have this hidden by default. It is probably tricky given that this is using {for} and it's related js for showing/hiding. We might need to add a hidden-by-default class or something like that.
Summary: ajaxify warning box on start page → fix flickering of warning box on start page
(Assignee)

Updated

7 years ago
OS: Mac OS X → All
Hardware: x86 → All
(Reporter)

Comment 2

7 years ago
Okay, then maybe deferred display: that box can be wrapped in it's own class, hidden by default and displayed after page load is complete. Would even be more effective ,if it would pop up, flash a bright color and fade to current color.
(Assignee)

Updated

7 years ago
Assignee: nobody → rrosario
(Assignee)

Comment 3

7 years ago
https://github.com/jsocol/kitsune/commit/1177f6859ef25ccebf577605a46646ae8bb53281
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: 2011Q4 → 2011-10-11
(Reporter)

Comment 4

7 years ago
So, what solution did you go with? In the check-in it looks like the box is set to display: none. Is there accompanying JS to switch it on after the page is loaded? I'm just asking because I can still see the flicker on staging.
(Assignee)

Comment 5

7 years ago
(In reply to Kadir Topal [:atopal] from comment #4)
> So, what solution did you go with? In the check-in it looks like the box is
> set to display: none. Is there accompanying JS to switch it on after the
> page is loaded? I'm just asking because I can still see the flicker on
> staging.

Yes, the SHOWFOR script will reset the display appropriately. If you are seeing it after clearing cache, this may not have worked then :(.
(Assignee)

Comment 6

7 years ago
Yeah, I see it on stage. I will try something else...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 2011-10-11 → 2011-10-18
(Assignee)

Comment 7

7 years ago
https://github.com/jsocol/kitsune/commit/770ab6d2467d79072ad4465778ed85ca1f007a0c
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
(Reporter)

Comment 8

7 years ago
Awesome, no more flickering :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.