After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 737425 - Overlay the spinner instead of switching to a different view
Overlay the spinner instead of switching to a different view
Status: RESOLVED FIXED
Product: gnome-photos
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME photos maintainer(s)
GNOME photos maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-09-26 12:03 UTC by Debarshi Ray
Modified: 2014-09-26 16:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
embed, spinner-box: Overlay the spinner (6.07 KB, patch)
2014-09-26 12:28 UTC, Debarshi Ray
committed Details | Review
preview-nav-buttons: Skip invalid paths (1.28 KB, patch)
2014-09-26 15:06 UTC, Debarshi Ray
committed Details | Review

Description Debarshi Ray 2014-09-26 12:03:13 UTC
The backgrounds in the overview and preview have a texture. Therefore when we switch the view to show a spinner with an opaque background, we get a splash. ie. texture -> opaque -> texture. This looks bad. Instead, let's overlay the spinner and not use an opaque background.
Comment 1 Debarshi Ray 2014-09-26 12:28:07 UTC
Created attachment 287151 [details] [review]
embed, spinner-box: Overlay the spinner
Comment 2 Debarshi Ray 2014-09-26 15:06:51 UTC
Created attachment 287170 [details] [review]
preview-nav-buttons: Skip invalid paths