Black backround image -CrossFadeImage does not set the sourceSize for the images
Bug #1227783 reported by
Günter Schwann
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu UI Toolkit |
Fix Released
|
Critical
|
Tim Peeters | ||
ubuntu-system-settings (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Saucy |
Invalid
|
Undecided
|
Unassigned | ||
ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
Critical
|
Tim Peeters | ||
Saucy |
Fix Released
|
Critical
|
Tim Peeters | ||
unity8 (Ubuntu) |
Fix Released
|
Undecided
|
Michael Zanetti | ||
Saucy |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Since image 20130918 it's possible to set the background image in the settings app.
There one can see a preview of the image using a CrossFadeImage component
When selecting a new image from the gallery-app that exceeds the max GL texture size (2048 on maguro for example), the image is black. If the sourceSize would be set, the image would be visible and less resouces would be used.
The unity8 shell uses a copy of that component (and will maybe move to the SDK one) and has the same issue.
Related branches
lp:~mzanetti/unity8/drop-crossfadeimage
- Michał Sawicz: Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 322 lines (+2/-278)5 files modifiedComponents/CrossFadeImage.qml (+0/-151)
Greeter/GreeterContent.qml (+1/-1)
Shell.qml (+1/-0)
tests/qmltests/CMakeLists.txt (+0/-1)
tests/qmltests/Components/tst_CrossFadeImage.qml (+0/-125)
lp:~tpeeters/ubuntu-ui-toolkit/crossFadeImage_SourceSize_fix
- PS Jenkins bot: Approve (continuous-integration)
- Tim Peeters: Approve
- Zsombor Egri: Needs Fixing
- Günter Schwann (community): Approve
- Michael Zanetti (community): Needs Information
-
Diff: 143 lines (+71/-2)4 files modifiedCHANGES (+1/-0)
components.api (+1/-1)
modules/Ubuntu/Components/CrossFadeImage.qml (+43/-1)
tests/unit/tst_components/tst_CrossFadeImage.qml (+26/-0)
lp:~zsombi/ubuntu-ui-toolkit/crossFadeImage_SourceSize_fix
Rejected
for merging
into
lp:ubuntu-ui-toolkit
- Tim Peeters: Disapprove
- PS Jenkins bot: Needs Fixing (continuous-integration)
- Zoltan Balogh: Approve
-
Diff: 120 lines (+55/-2)4 files modifiedCHANGES (+1/-0)
components.api (+1/-1)
modules/Ubuntu/Components/CrossFadeImage.qml (+41/-1)
tests/unit/tst_components/tst_CrossFadeImage.qml (+12/-0)
description: | updated |
summary: |
- CrossFadeImage does not set the sourceSize for the images + Black backround image -CrossFadeImage does not set the sourceSize for + the images |
Changed in ubuntu-ui-toolkit (Ubuntu Saucy): | |
assignee: | nobody → Tim Peeters (tpeeters) |
importance: | Undecided → High |
Changed in ubuntu-ui-toolkit (Ubuntu Saucy): | |
importance: | High → Critical |
Changed in ubuntu-ui-toolkit: | |
status: | New → Confirmed |
status: | Confirmed → In Progress |
Changed in unity8: | |
status: | Fix Committed → Fix Released |
Changed in ubuntu-ui-toolkit (Ubuntu Saucy): | |
status: | New → In Progress |
Changed in ubuntu-ui-toolkit: | |
status: | Fix Committed → Fix Released |
Changed in ubuntu-ui-toolkit (Ubuntu Saucy): | |
status: | In Progress → Fix Committed |
Changed in ubuntu-ui-toolkit (Ubuntu Saucy): | |
status: | Fix Committed → Fix Released |
Changed in unity8 (Ubuntu): | |
assignee: | nobody → Michael Zanetti (mzanetti) |
no longer affects: | unity8 |
To post a comment you must log in.
is there a small test program to reproduce the issue?