First page disappears, other pages shift

Bug #952707 reported by Sergey "Shnatsel" Davidoff
48
This bug affects 9 people
Affects Status Importance Assigned to Milestone
Slingshot
Fix Released
High
Andrea Basso

Bug Description

Sometimes first page of apps in Slingshot disappears, and by clicking "1" button I get the second page, by clicking "2" I get third page, etc. Last page is still shown but empty in this case.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :
Revision history for this message
Victor Martinez (victored) wrote :

Happens here too.

Changed in slingshot:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Moreover, when it happens, my first page simply disappears. Marking high priority because of that.

Changed in slingshot:
importance: Low → High
summary: - Slingshot sometimes shows empty pages
+ Firts page disappears, other pages shift
description: updated
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote : Re: Firts page disappears, other pages shift

According to duplicate bug 956068, this happens after installing an app.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I have this bug after updating packages but not installing any new ones

summary: - Firts page disappears, other pages shift
+ First page disappears, other pages shift
Revision history for this message
tim schoen (timschoen123-deactivatedaccount) wrote :

I have this for removing and updating. installing was a mistake, I was installing apps and updating at the same time :p.

Revision history for this message
tim schoen (timschoen123-deactivatedaccount) wrote :

i think i found a fix but it will need glib and i don't know how to add --pkg flags to cmake.
the fix is: remove the following:

                categories = app_system.get_categories ();
                apps = app_system.get_apps ();

                populate_grid_view ();

and add this:

spawn_command_line_async ("killall slingshot");

could someone please explain how to add --pkg flags to cmake or try this themselves?

Revision history for this message
xapantu (xapantu) wrote : Re: [Bug 952707] Re: First page disappears, other pages shift

All is in CMakeLists.txt files. But you don't need to add glib, it is
added since gtk (and granite, and everything related to vala (if we
don't count sdl)).

I'm not sure doing a "killall slingshot" is really the right way to do
it, we should be able to make a clean reloading/adding without having to
close the process.

Le 15/03/2012 17:59, tim schoen a écrit :
> i think i found a fix but it will need glib and i don't know how to add --pkg flags to cmake.
> the fix is: remove the following:
>
> categories = app_system.get_categories ();
> apps = app_system.get_apps ();
>
> populate_grid_view ();
>
> and add this:
>
> spawn_command_line_async ("killall slingshot");
>
> could someone please explain how to add --pkg flags to cmake or try this
> themselves?
>

Revision history for this message
Cassidy James Blaede (cassidyjames) wrote :

Yeah, I'm with Lucas; making Slingshot commit suicide to workaround a legit
bug just seems hacky. ;)

--
Sent from *elementary OS* <http://elementaryos.org/>.

On Thu, Mar 15, 2012 at 12:15 PM, xapantu <email address hidden> wrote:

> All is in CMakeLists.txt files. But you don't need to add glib, it is
> added since gtk (and granite, and everything related to vala (if we
> don't count sdl)).
>
> I'm not sure doing a "killall slingshot" is really the right way to do
> it, we should be able to make a clean reloading/adding without having to
> close the process.
>
> Le 15/03/2012 17:59, tim schoen a écrit :
> > i think i found a fix but it will need glib and i don't know how to add
> --pkg flags to cmake.
> > the fix is: remove the following:
> >
> > categories = app_system.get_categories ();
> > apps = app_system.get_apps ();
> >
> > populate_grid_view ();
> >
> > and add this:
> >
> > spawn_command_line_async ("killall slingshot");
> >
> > could someone please explain how to add --pkg flags to cmake or try this
> > themselves?
> >
>
> --
> You received this bug notification because you are a member of
> elementary Pantheon team, which is the registrant for Slingshot.
> https://bugs.launchpad.net/bugs/952707
>
> Title:
> First page disappears, other pages shift
>
> Status in Slingshot:
> Confirmed
>
> Bug description:
> Sometimes first page of apps in Slingshot disappears, and by clicking
> "1" button I get the second page, by clicking "2" I get third page,
> etc. Last page is still shown but empty in this case.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/slingshot/+bug/952707/+subscriptions
>

Revision history for this message
tim schoen (timschoen123-deactivatedaccount) wrote :

I think it will work pretty well. It means you can replace 3 lines of code for 1 line of code and fix a bug. I think there aren't many other ways to fix this bug cause the code there is now already does reload it in the "right" way, and that doesn't seem to work :p.

Changed in slingshot:
milestone: none → luna-beta1
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

Do you already have this bug? For me it is fixed

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Haven't been noticing it lately, let's see if anybody confirms it now

Changed in slingshot:
status: Confirmed → Incomplete
Revision history for this message
Mario Guerriero (mefrio-g) wrote :

I have to confirm this bug again. Installing a new app, first page disappears

Revision history for this message
Jacek Spławski (zacol1) wrote :

This bug is still actually.

Revision history for this message
Andrea Basso (voluntatefaber) wrote :

I can confirm, bug is still present.
In the upcoming days I'm gonna take a look at it (even though I'm quite busy, likel from Saturday). If there's a simple fix let's use it, which is the best. But if we cannot find any way to fix it (and seems as illogical as the close-on-focus-out bug) I propose to simply use Tim's fix for Luna and find a proper solution for L+1.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

This wasn't an issue in Clutter-based Slingshot. Maybe we should revive that branch?

Changed in slingshot:
status: Incomplete → Confirmed
Revision history for this message
Danielle Foré (danrabbit) wrote :

Clutter Slingshot has a plethora of other rather show-stopping bugs. If a proper fix isn't simple, I don't think it's entirely evil to use Tim's fix as a temporary solution.

We need to be operating as if we have very limited resources and we're trying to make a release before Summer has come and gone. There will be plenty of time for major changes in the Luna +1 cycle, let's just focus on making things work for now.

Changed in slingshot:
assignee: nobody → Andrea Basso (voluntatefaber)
Changed in slingshot:
status: Confirmed → Fix Committed
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I still get this bug fairly reliably in version 0.6-0~r221+pkg10~precise1
It's not related to software updates this time - all I have to do is to scroll back and forth quickly.

Changed in slingshot:
status: Fix Committed → Confirmed
Revision history for this message
Andrea Basso (voluntatefaber) wrote :

Shnatsel, I can't reproduc it, can you please make a screencast?

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Here you go. Took a long time to get it this time.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I also got it to shift by just one icon row on a slow machine.

Revision history for this message
Andrea Basso (voluntatefaber) wrote :

I tried everything but I can't reproduce it on my machine. I pushed lp:~voluntatefaber/slingshot/page-shifting with some more checking before moving a page. Can you please test and tell me if still happens?

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Thanks! I cannot build the merge of this branch with lp:slingshot though, here's the error I get:

/home/precise/sandbox/slingshot/src/SlingshotView.vala:295.17-295.82: error: 1 extra arguments for `void Granite.Widgets.PopOver.move_to_coords (int x, int y)'
                move_to_coords (monitor_dimensions.x, monitor_dimensions.y, false); // this would be coordinates 0,0 on the screen
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Revision history for this message
Andrea Basso (voluntatefaber) wrote :

That's because of a recent change in granite. But lp:~voluntatefaber/slingshot/page-shifting should build fine with older granite.

Changed in slingshot:
status: Confirmed → Fix Committed
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Happens again in version 0.6+r283-0+pkg13~precise1 and possibly a few earlier ones

Changed in slingshot:
status: Fix Committed → Confirmed
Revision history for this message
Andrea Basso (voluntatefaber) wrote :

Shoot! I wanted to release Slingshot by tomorrow but if this is back in action we can't! And not having a way to reproduce it make tracking it even harder!

Revision history for this message
Andrea Basso (voluntatefaber) wrote :

After some testing, I saw it's a bug introduced with the not-too tested gmenu-3 revision. I'm switching back to the old gmenu for beta1 and hope to find a solution with Ricotz by beta2 or luna release.

Changed in slingshot:
status: Confirmed → Fix Committed
Revision history for this message
Santiago Ocamica (santi698) wrote :

I could reproduce this bug in the last build from elementary daily doing the following:
  1. Start to install an app that puts an icon in slingshot.
  2. Open slingshot and start switching pages.
  3. Just before the application ends installing slingshot has all the icons shifted.
I attach a screencast.
Hope this helps.

Changed in slingshot:
status: Fix Committed → Fix Released
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

I'm reopening the bug based on Shanti's comment.

Changed in slingshot:
milestone: 0.7 → none
status: Fix Released → New
Changed in slingshot:
status: New → Confirmed
Changed in slingshot:
milestone: none → luna-beta2
Changed in slingshot:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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