Ticket #10539 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Gnome: users can remove top panel

Reported by: godiard Owned by: martin.langhoff
Priority: normal Milestone: 10.1.3
Component: gnome-desktop Version: not specified
Keywords: Cc: martin.langhoff, sridhar
Action Needed: no action Verified: no
Deployments affected: Blocked By:
Blocking: #10409

Description

If the user remove the top panel can't start applications / change NetworkManager connections, etc

Attachments

Change History

Changed 3 years ago by godiard

  • next_action changed from never set to code

We can detect the it the top panel was removed doing:

gconftool-2 -g /apps/panel/general/toplevel_id_list

if the panel was removed returns:

[bottom_panel]

if not

[top_panel,bottom_panel]

to get the default panels again, we can do:

gconftool-2 --recursive-unset /apps/panel

The bottom panel can't be removed.

I need to see where place this code.

Changed 3 years ago by pgf

why do you say the bottom panel can't be removed? (i just did it.)

Changed 3 years ago by godiard

Because i tried after removing the top panel...

Now tried removing the bottom panel first, and is possible.

Conclusion, the last panel can't be removed.

Changed 3 years ago by godiard

  • cc martin.langhoff added
  • owner set to martin.langhoff
  • next_action changed from code to review
  • component changed from not assigned to gnome-desktop

Agreed with martin_xsa, we will check for existence of systray, window_list and menu_bar

We need add to /usr/bin/olpc-session in the Non Sugar Desktop section:

# Check existence of  systray, window_list and menu_bar applets
fix_gnome=false
if ! /usr/bin/gconftool-2 -g /apps/panel/general/applet_id_list | grep -q systray; 
then 
    fix_gnome=true
fi
if ! /usr/bin/gconftool-2 -g /apps/panel/general/applet_id_list | grep -q window_list;
then 
    fix_gnome=true 
fi
if ! /usr/bin/gconftool-2 -g /apps/panel/general/object_id_list | grep -q menu_bar;
then 
    fix_gnome=true
fi
if [ $fix_gnome = true ] ; then
    echo "ARREGLAR"
    gconftool-2 --recursive-unset /apps/panel
fi

Changed 3 years ago by martin.langhoff

  • next_action changed from review to add to build

Patch, with minor tweaks, in olpc-utils 1.0.36 - awaiting next build... thanks Gonzalo!

Changed 3 years ago by erikos

  • next_action changed from add to build to test in build

Is in 359.

Changed 3 years ago by godiard

|TestCase| Remove Main Menu, sys tray or window list, or the panels containing them, reboot X (or go to Sugar and return to Gnome) and the default panels will be restored.

Changed 3 years ago by greenfeld

  • status changed from new to closed
  • next_action changed from test in build to no action
  • resolution set to fixed

Removing the Window list, Start Menu, and/or the panels containing these items as subitems is detected and corrected by restoring all these to the default settings. Tested in 10.1.3 os360.

Of note:

  • You can delete a panel without resetting everything if it does not contain an item we consider critical. You can also control which screen border a panel is on safely.
  • We cannot detect if someone deletes an item we consider critical and then adds it in a different spot. This forces a restore. Users need to move items (unlocking it if necessary from the same right-click menu used to allow moving) instead.
  • Alternatives items (such as using the Window Selector instead of Window List) which might be safe currently are not supported.

Changed 3 years ago by sridhar

  • cc sridhar added
Note: See TracTickets for help on using tickets.