Add feedback when opening VeraCrypt Mounter
#4 Updated by segfault 2019-01-10 12:08:54
- Assignee changed from segfault to sajolida
- QA Check changed from Ready for QA to Dev Needed
> Hi segfault! Do you want to take this one?
> Optimistically assigning
Bug #16333 to you as well since it is similar
> Here is a patch.
It doesn’t seem to work or I don’t understand what it’s supposed to do. I tested it by adding the two lines to the desktop launcher in Tails (which is called unlock-veracrypt-volumes and not veracrypt-mounter) and then starting the app. I expected the mouse cursor to spin until the window is shown, but the cursor didn’t change. I see a spinner in the top bar but that is also present without the two lines. I made sure that when I start the app it does indeed use the modified desktop launcher (tested by changing the
#5 Updated by sajolida 2019-01-12 17:38:09
> I don’t understand what it’s supposed to do.
Yes, It’s supposed to change the mouse cursor into a spinner right after
choosing to start the app. Since VeraCrypt Mounter opens super fast,
it’s only visible for less than a second on my machine (but I guess it
could take longer in worse conditions).
> I made sure that when I start the app it does indeed use the modified desktop launcher (tested by changing the
Did you also restart GNOME Shell? I need to do this for my changes to be
applied on Tails:
Alt + F2 + typing “r” or “gnome-shell —replace”
#7 Updated by segfault 2019-01-18 10:30:00
- Feature Branch set to feature/16334-feedback-when-opening-veracrypt
> Did you also restart GNOME Shell? I need to do this for my changes to be
> applied on Tails:
> Alt + F2 + typing “r” or “gnome-shell —replace”
This worked, thanks for the hint.
I applied the patch and am now building the branch to test it. Maybe this can still make it into 3.12.
#13 Updated by intrigeri 2019-02-11 08:59:50
- Status changed from Fix committed to In Progress
- Assignee changed from segfault to intrigeri
- % Done changed from 0 to 90
3.13 won’t be built from devel and there’s a chance we have no major release for a long while, so I’ll consider cherry-picking this into stable later this week.