"Repository disk usage" section flashes green when PPA page loads

Bug #338247 reported by Matthew Paul Thomas
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
Celso Providelo

Bug Description

1. Go to a PPA page, such as <https://launchpad.net/~ted-gould/+archive/ppa>.

What happens: Every time the page loads, the "Repository disk usage" section flashes green.
What should happen: The section should not flash, because it's not changing from a previous value.

Tags: lp-soyuz ppa ui
description: updated
Celso Providelo (cprov)
Changed in soyuz:
milestone: none → pending
status: New → Triaged
Revision history for this message
Julian Edwards (julian-edwards) wrote :

This is easy to fix, so let's do it!

Changed in soyuz:
assignee: nobody → cprov
importance: Undecided → Medium
milestone: pending → 2.2.3
Revision history for this message
Celso Providelo (cprov) wrote :

Matthew,

Theoretically the repository-size values could have changed since the last loading, but that's not really the point about green-flashing.

We should removing the flashing because the hypothetical change was not triggered by the user, if it existed it was automatically executed.

I suspect that same action should be taken for the expandable rows (package extra information) in the PPA page, they are just loaded dynamically, not "successfully modified by the user".

Do you agree ?

Revision history for this message
Julian Edwards (julian-edwards) wrote : Re: [Bug 338247] Re: "Repository disk usage" section flashes green when PPA page loads

More to the point, it will become consistent with other
dynamically-loaded parts of pages (e.g. the subscribers on the bugs
page) which don't flash.

With regard to the expandable rows part, Martin A. did a "slider" for
those I believe, so I'd like to see that there instead if he's finished
it? (it was working at our sprint in SC)

Celso Providelo (cprov)
Changed in soyuz:
status: Triaged → In Progress
Revision history for this message
Celso Providelo (cprov) wrote :

RF 7934

Changed in soyuz:
status: In Progress → Confirmed
status: Confirmed → Fix Committed
Celso Providelo (cprov)
Changed in soyuz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.