Launcher - clicking trash or device icons multiple times opens multiple instances
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Ayatana Design |
High
|
John Lea | ||
| Nautilus |
Fix Released
|
Medium
|
||
| Unity |
Won't Fix
|
High
|
Marco Trevisan (Treviño) | |
| unity-2d |
Medium
|
Robert Sajdok | ||
| bamf (Ubuntu) |
High
|
Unassigned | ||
| nautilus (Ubuntu) |
Low
|
Marco Trevisan (Treviño) | ||
| unity (Ubuntu) |
High
|
Marco Trevisan (Treviño) | ||
| unity-2d (Ubuntu) |
Undecided
|
Unassigned |
Bug Description
When you click on the trash icon multiple times, it will keep opening new windows.
Instead it should bring the open window to focus. I can not think of any circumstances where two trash windows are necessary. But in case it ever is, a "Open new window" entry should be in it's quicklist.
-------
Desired Solution:
- If a 'trash' Nautilus window is already open, clicking on the Trash Launcher icon should focus the already open Nautilus window that is currently displaying the trash
- However 'window open' or 'running' triangles *should not* be displayed (on either side of the Trash icon)
Related branches
- Alex Launi (community): Needs Fixing on 2011-05-05
- Mirco Müller (community): Approve on 2011-02-17
- Jason Smith (community): Approve on 2011-02-17
-
Diff: 178 lines (+65/-15)3 files modifiedsrc/LauncherController.cpp (+2/-2)
src/TrashLauncherIcon.cpp (+53/-11)
src/TrashLauncherIcon.h (+10/-2)
- Florian Boucault (community): Needs Fixing (code) on 2011-04-28
-
Diff: 327 lines (+148/-62)7 files modifiedlauncher/UnityApplications/CMakeLists.txt (+2/-0)
launcher/UnityApplications/launcherapplication.cpp (+2/-56)
launcher/UnityApplications/launcherapplication.h (+0/-2)
launcher/UnityApplications/launcherutility.cpp (+58/-0)
launcher/UnityApplications/launcherutility.h (+16/-0)
launcher/UnityApplications/trash.cpp (+66/-2)
launcher/UnityApplications/trash.h (+4/-2)
- Aurélien Gâteau: Pending requested 2011-03-22
- Diff: 0 lines
- Unity Team: Pending requested 2013-01-15
-
Diff: 209 lines (+137/-4) (has conflicts)3 files modifiedlauncher/LauncherController.cpp (+56/-0)
launcher/TrashLauncherIcon.cpp (+64/-4)
launcher/TrashLauncherIcon.h (+17/-0)
- Ubuntu Desktop: Pending requested 2013-03-12
-
Diff: 555 lines (+366/-34)6 files modifieddebian/changelog (+10/-1)
debian/patches/10_reuse_already_opened_views.patch (+302/-0)
debian/patches/12_unity_launcher_support.patch (+35/-28)
debian/patches/17_static_unity_quicklist.patch (+5/-5)
debian/patches/19_add_desktop_keywords.patch (+12/-0)
debian/patches/series (+2/-0)
Changed in unity: | |
status: | New → Confirmed |
Mark Shuttleworth (sabdfl) wrote : Re: [Bug 692444] [NEW] clicking trash multiple times opens multiple instances of it. | #1 |
tags: | added: bitesize |
Changed in unity-2d: | |
status: | New → Confirmed |
Changed in unity: | |
importance: | Undecided → Medium |
assignee: | nobody → Jason Smith (jassmith) |
milestone: | none → 3.6 |
Changed in unity-2d: | |
milestone: | none → 3.6 |
importance: | Undecided → Medium |
Assigning to Jason by default, but the bug is still up for grab as a bitesize one.
Changed in unity-2d: | |
milestone: | 3.6 → 3.8 |
Changed in unity: | |
status: | Confirmed → In Progress |
assignee: | Jason Smith (jassmith) → Stefano Candori (cando) |
Changed in unity: | |
milestone: | 3.6 → 3.4.2 |
Carl Simpson (cwd-simpson) wrote : | #3 |
If you have a trash window open, and it is already focussed, you might not notice- especially if you have a lot of Nautilus windows open at once (or you are just being scatterbrained, which it is so easy to be when handling a lot of information on a computer).
The proposed plan would mean that the trash button does nothing if the trash window is already in focus. You wouldn't think pressing the button had done anything at all, and might think that it is broken, for a moment. Maybe pressing the button could also highlight the window in some way? (But only when the window is already in focus?)
It seems bad for a piece of interface to on occasion do nothing whatsoever.
Cameron Hart (bitshifternz) wrote : | #4 |
Also if you open a trash window by clicking on the trash icon, what happens if the user navigates away from the trash folder in nautilus? Is is still a trash window at this point or a nautilus window? Should it raise the original window that was opened or a new one? Or does it raise any Nautilius window that is browsing the trash folder at the time?
It seems like something that could lead to quite inconsistent behaviour from a user perspective.
Changed in unity: | |
status: | In Progress → Confirmed |
Changed in bamf: | |
status: | New → Confirmed |
Changed in bamf (Ubuntu): | |
status: | New → Confirmed |
Changed in unity-2d: | |
milestone: | 3.8 → none |
Wade Pedersen (wmpedersen) wrote : | #5 |
@Cameron Perhaps I don't understand your meaning, but Trash always opens in the Nautilus launcher, and does not provide an indicator under Trash.
Changed in unity: | |
milestone: | 3.4.2 → 3.6.4 |
Changed in unity: | |
milestone: | 3.6.4 → 3.6.6 |
Balassa Máté (balassa-mm) wrote : | #6 |
What should happen if there are multiple instances open of the same trash/rubbish bin? Wich one will get the focus? This can be a source of other bugs...
On the other hand I've got to agree, it is much more user friendly if it is like imagined in the report...
Balassa Máté (balassa-mm) wrote : | #7 |
This was marked as a bitsize bug so while trying to help unity I'm trying to do something not so important...
I can not guarantee that I'm doing the right thing, this is my first try.
Changed in unity-2d: | |
assignee: | nobody → Matthew Balassa (balassa-mm) |
Changed in unity-2d: | |
assignee: | Matthew Balassa (balassa-mm) → nobody |
Changed in unity: | |
milestone: | 3.6.6 → 3.6.8 |
tags: | added: delta-with-3d |
Marco Biscaro (marcobiscaro2112) wrote : | #8 |
@Florian: delta-with-3d? But both Unity 2D and Unity 3D are affected by this bug.
Florian Boucault (fboucault) wrote : | #9 |
@Marco: I know but it seems there is a branch ready to land in Unity 3D that will fix the issue. I am planning ahead :)
Changed in unity: | |
milestone: | 3.6.8 → 3.8 |
Changed in unity: | |
milestone: | 3.8 → 3.8.2 |
Changed in unity: | |
milestone: | 3.8.2 → 3.8.4 |
Changed in unity: | |
milestone: | 3.8.4 → 3.8.6 |
Changed in unity: | |
milestone: | 3.8.6 → 3.8.8 |
Richard Dale (rdale) wrote : | #10 |
I've been looking into this, and at the moment the trash file manager is launched with a g_app_info_
A possible work round would be to simply have a desktop file called 'trash.desktop' with the trashcan icon, that had a line in it 'Exec: nautilus 'trash://'. That way the application would only be launched once as it would be just a normal app icon rather than something special cased.
Changed in unity-2d: | |
assignee: | nobody → Robert Sajdok (ris) |
status: | Confirmed → In Progress |
Changed in unity: | |
milestone: | 3.8.8 → 3.8.10 |
Olivier Tilloy (osomon) wrote : | #11 |
Interesting suggestion Richard! However, for it to be really portable the desktop file should not invoke nautilus directly (the user may have another file manager of choice), but rather xdg-open. `xdg-open trash://` should just do the right thing. Alas, I seriously doubt the launcher, through bamf, will be able to match whatever window opens as a result to the trash. It would be worth a try though, there may be an elegant solution to this problem that I’m overlooking.
Also note how the implementation proposed by Stefano for unity addresses the problem (have a look at the pending merge request, it contains detailed explanations). It does have a weak point, but it’s an original way of tackling the problem, you may be able to get inspiration from it and perfect it ;)
Changed in unity-2d: | |
milestone: | none → 3.10 |
Changed in unity: | |
milestone: | 3.8.10 → 3.8.12 |
David Barth (dbarth) wrote : | #12 |
The workaround doesn't feel safe to land in an SRU anymore. I'd rather have the logic of that go outside of the window manager, where it can be stabilized without risking the whole rest of the process.
Changed in unity: | |
milestone: | 3.8.12 → backlog |
Changed in unity-2d: | |
status: | In Progress → Fix Committed |
Changed in bamf: | |
status: | Confirmed → Fix Committed |
Changed in bamf (Ubuntu): | |
status: | Confirmed → Fix Committed |
Changed in unity: | |
status: | Confirmed → Fix Committed |
Marco Biscaro (marcobiscaro2112) wrote : | #13 |
The proposed branch was not merged with trunk. Reverting to Confirmed.
Changed in unity: | |
status: | Fix Committed → Confirmed |
Changed in unity: | |
status: | Confirmed → Fix Committed |
Changed in unity-2d: | |
milestone: | 3.10 → 3.8.6 |
Changed in unity-2d: | |
status: | Fix Committed → Fix Released |
Changed in bamf: | |
status: | Fix Committed → Fix Released |
Changed in unity: | |
status: | Fix Committed → Fix Released |
Changed in bamf (Ubuntu): | |
status: | Fix Committed → Fix Released |
Changed in unity: | |
assignee: | Stefano Candori (cando) → nobody |
Changed in bamf: | |
status: | Fix Released → Confirmed |
Changed in unity: | |
status: | Fix Released → Confirmed |
affects: | bamf (Ubuntu) → unity (Ubuntu) |
Changed in unity (Ubuntu): | |
importance: | Undecided → Medium |
status: | Fix Released → Confirmed |
Victor Martinez (victored) wrote : | #14 |
I think that implementing a Trash Place would be a better solution to this bug.
Changed in bamf (Ubuntu): | |
status: | New → Confirmed |
Changed in bamf: | |
status: | Confirmed → Fix Released |
Changed in unity (Ubuntu): | |
status: | Confirmed → Triaged |
summary: |
- clicking trash multiple times opens multiple instances of it. + Launcher - clicking trash or device icons multiple times opens multiple + instances |
Changed in unity: | |
assignee: | nobody → Brandon Schaefer (brandontschaefer) |
status: | Confirmed → In Progress |
description: | updated |
tags: | added: udp |
Changed in ayatana-design: | |
assignee: | nobody → John Lea (johnlea) |
importance: | Undecided → Medium |
importance: | Medium → High |
status: | New → Triaged |
Changed in unity: | |
status: | In Progress → Confirmed |
assignee: | Brandon Schaefer (brandontschaefer) → nobody |
Changed in ayatana-design: | |
status: | Triaged → Fix Committed |
Changed in unity: | |
status: | Confirmed → Triaged |
Changed in bamf (Ubuntu): | |
status: | Confirmed → Triaged |
Launchpad Janitor (janitor) wrote : | #15 |
Status changed to 'Confirmed' because the bug affects multiple users.
Changed in unity-2d (Ubuntu): | |
status: | New → Confirmed |
Changed in unity: | |
milestone: | backlog → none |
tags: | added: exbacklog |
no longer affects: | unity-2d (Ubuntu) |
Changed in bamf: | |
status: | Fix Released → Triaged |
Changed in unity (Ubuntu): | |
importance: | Medium → High |
Changed in bamf (Ubuntu): | |
importance: | Undecided → High |
Changed in unity: | |
importance: | Medium → High |
Changed in bamf: | |
importance: | Undecided → High |
Changed in unity-2d (Ubuntu): | |
status: | New → Confirmed |
Changed in nautilus (Ubuntu): | |
status: | New → In Progress |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
no longer affects: | unity-2d (Ubuntu) |
no longer affects: | bamf |
Changed in bamf (Ubuntu): | |
status: | Triaged → Invalid |
Changed in unity-2d (Ubuntu): | |
status: | New → Confirmed |
Changed in unity: | |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
Stephen M. Webb (bregma) wrote : | #18 |
Evidently this was fixed in Unity-2D some time ago but not properly marked in the release.
Changed in unity-2d (Ubuntu): | |
status: | Confirmed → Fix Released |
Changed in unity (Ubuntu): | |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
Changed in nautilus: | |
importance: | Unknown → Medium |
status: | Unknown → In Progress |
Changed in nautilus: | |
status: | In Progress → Fix Released |
Changed in unity: | |
status: | Triaged → Won't Fix |
Changed in unity (Ubuntu): | |
status: | Triaged → Invalid |
Changed in nautilus (Ubuntu): | |
importance: | Undecided → Low |
Launchpad Janitor (janitor) wrote : | #19 |
This bug was fixed in the package nautilus - 1:3.6.3-0ubuntu9
---------------
nautilus (1:3.6.3-0ubuntu9) raring; urgency=low
* Renamed patches coming from the upstream vcs to be prefixed with git_
* debian/
- use --new-window in that desktop as well
[ Marco Trevisan (Treviño) ]
* debian/
- Try to reuse the available windows or tabs when opening an already opened
location (lp: #692444)
* debian/
- Quicklist entries to show the copy dialog should use a separator (lp: #1154109)
- Transfer dialog should be presented using the correct timestamp (lp: #1154111)
- Use the quicklist activation timestamp to set the application user time
so that the opened windows will be focused (lp: #1154237)
* debian/
- Use nautilus --new-window to open a New window
* debian/
- Adding .desktop files keywords from upstream
* debian/
- Export the opened locations to dbus (lp: #1063830)
-- Sebastien Bacher <email address hidden> Thu, 14 Mar 2013 11:23:18 +0100
Changed in nautilus (Ubuntu): | |
status: | In Progress → Fix Released |
Changed in ayatana-design: | |
status: | Fix Committed → Fix Released |
status confirmed
The Trash window should be re-focused by default, and yes, a quicklist
item for "Open the Rubbish Bin again" would be good.
Mark