Merge lp:~shnatsel/slingshot/app-visibility-fixes into lp:~elementary-pantheon/slingshot/trunk

Proposed by Sergey "Shnatsel" Davidoff
Status: Merged
Approved by: Andrea Basso
Approved revision: 225
Merged at revision: 226
Proposed branch: lp:~shnatsel/slingshot/app-visibility-fixes
Merge into: lp:~elementary-pantheon/slingshot/trunk
Diff against target: 218 lines (+41/-123)
1 file modified
data/pantheon-applications.menu (+41/-123)
To merge this branch: bzr merge lp:~shnatsel/slingshot/app-visibility-fixes
Reviewer Review Type Date Requested Status
Andrea Basso (community) Approve
Review via email: mp+111860@code.launchpad.net

Commit message

Fixes multiple visibility issues.

Description of the change

Fixes multiple visibility issues:
- Some apps like usb-creator-gtk or synaptic not being displayed
- Some games not being displayed
- Some crap like xterm being shown

To post a comment you must log in.
Revision history for this message
Andrea Basso (voluntatefaber) wrote :

This is great and fixes both bugs.
With this new branch, something strange happened. I've created two custom launchers with Alacarte to test its support in Slingshot and -previosuly- didn't appear. Now, they appear only in Category View and under "Other", no matter which category I'd set.
Do you think it's possible to make them appear in all three views and in the right category? If that's not possible, is at least possible not to show them at all?
In the meantime, I'm gonna merge this because it both works great and fixes what it's supposed to.

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

I've just found out that all the apps that are listed under System Tools > Preferences in Alacarte show up under Other. Is this by design or they're supposed to be under System?

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

Alacarte is way too bound to GNOME2 category layout. I don't think it's realistic to support it fully.

Still, I guess it should be possible to support creating launchers. Let's open a bug about that. I have no idea about not showing something in the main view, but if it's possible, I'd be glad to use that bug-o-feature for accessibility apps.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/pantheon-applications.menu'
2--- data/pantheon-applications.menu 2012-03-22 15:37:53 +0000
3+++ data/pantheon-applications.menu 2012-06-25 14:34:34 +0000
4@@ -24,17 +24,14 @@
5 <Include>
6 <And>
7 <Category>Utility</Category>
8- <!-- Accessibility spec must have either the Utility or Settings
9- category, and we display an accessibility submenu already for
10- the ones that do not have Settings, so don't display accessibility
11- applications here -->
12+ <!-- Accessibility spec must have either the Utility or Settings
13+ category, and we display an accessibility submenu already for
14+ the ones that do not have Settings, so don't display accessibility
15+ applications here -->
16 <Not><Category>Accessibility</Category></Not>
17 <Not><Category>System</Category></Not>
18 </And>
19 </Include>
20- <Exclude>
21- <Filename>plank.desktop</Filename>
22- </Exclude>
23 </Menu> <!-- End Accessories -->
24
25 <!-- Accessibility submenu -->
26@@ -92,107 +89,8 @@
27 <Include>
28 <And>
29 <Category>Game</Category>
30- <Not><Category>ActionGame</Category></Not>
31- <Not><Category>AdventureGame</Category></Not>
32- <Not><Category>ArcadeGame</Category></Not>
33- <Not><Category>BoardGame</Category></Not>
34- <Not><Category>BlocksGame</Category></Not>
35- <Not><Category>CardGame</Category></Not>
36- <Not><Category>KidsGame</Category></Not>
37- <Not><Category>LogicGame</Category></Not>
38- <Not><Category>Simulation</Category></Not>
39- <Not><Category>SportsGame</Category></Not>
40- <Not><Category>StrategyGame</Category></Not>
41 </And>
42 </Include>
43- <DefaultLayout inline="true" inline_limit="6" inline_header="false">
44- <Merge type="menus"/>
45- <Merge type="files"/>
46- </DefaultLayout>
47- <Menu>
48- <Name>Action</Name>
49- <Directory>ActionGames.directory</Directory>
50- <Include>
51- <Category>ActionGame</Category>
52- </Include>
53- </Menu>
54- <Menu>
55- <Name>Adventure</Name>
56- <Directory>AdventureGames.directory</Directory>
57- <Include>
58- <Category>AdventureGame</Category>
59- </Include>
60- </Menu>
61- <Menu>
62- <Name>Arcade</Name>
63- <Directory>ArcadeGames.directory</Directory>
64- <Include>
65- <Category>ArcadeGame</Category>
66- </Include>
67- </Menu>
68- <Menu>
69- <Name>Board</Name>
70- <Directory>BoardGames.directory</Directory>
71- <Include>
72- <Category>BoardGame</Category>
73- </Include>
74- </Menu>
75- <Menu>
76- <Name>Blocks</Name>
77- <Directory>BlocksGames.directory</Directory>
78- <Include>
79- <Category>BlocksGame</Category>
80- </Include>
81- </Menu>
82- <Menu>
83- <Name>Cards</Name>
84- <Directory>CardGames.directory</Directory>
85- <Include>
86- <Category>CardGame</Category>
87- </Include>
88- </Menu>
89- <Menu>
90- <Name>Kids</Name>
91- <Directory>KidsGames.directory</Directory>
92- <Include>
93- <Category>KidsGame</Category>
94- </Include>
95- </Menu>
96- <Menu>
97- <Name>Logic</Name>
98- <Directory>LogicGames.directory</Directory>
99- <Include>
100- <Category>LogicGame</Category>
101- </Include>
102- </Menu>
103- <Menu>
104- <Name>Role Playing</Name>
105- <Directory>RolePlayingGames.directory</Directory>
106- <Include>
107- <Category>RolePlaying</Category>
108- </Include>
109- </Menu>
110- <Menu>
111- <Name>Simulation</Name>
112- <Directory>SimulationGames.directory</Directory>
113- <Include>
114- <Category>Simulation</Category>
115- </Include>
116- </Menu>
117- <Menu>
118- <Name>Sports</Name>
119- <Directory>SportsGames.directory</Directory>
120- <Include>
121- <Category>SportsGame</Category>
122- </Include>
123- </Menu>
124- <Menu>
125- <Name>Strategy</Name>
126- <Directory>StrategyGames.directory</Directory>
127- <Include>
128- <Category>StrategyGame</Category>
129- </Include>
130- </Menu>
131 </Menu> <!-- End Games -->
132
133 <!-- Graphics -->
134@@ -202,6 +100,7 @@
135 <Include>
136 <And>
137 <Category>Graphics</Category>
138+ <Not><Filename>evince.desktop</Filename></Not>
139 </And>
140 </Include>
141 </Menu> <!-- End Graphics -->
142@@ -245,12 +144,18 @@
143 <Directory>System-Tools.directory</Directory>
144 <Include>
145 <And>
146- <Category>System</Category>
147- <Not><Category>Settings</Category></Not>
148- <Not><Category>Game</Category></Not>
149+ <Or>
150+ <Category>System</Category>
151+ <Category>Administration</Category>
152+ </Or>
153+ <Not>
154+ <Category>Game</Category>
155+ <Category>X-GNOME-Settings-Panel</Category>
156+ <Filename>debian-xterm.desktop</Filename>
157+ <Filename>debian-uxterm.desktop</Filename>
158+ <Filename>ibus.desktop</Filename>
159+ </Not>
160 </And>
161- <Filename>switchboard.desktop</Filename>
162- <Filename>ubuntu-software-center.desktop</Filename>
163 </Include>
164 </Menu> <!-- End System Tools -->
165
166@@ -261,14 +166,35 @@
167 <OnlyUnallocated/>
168 <Include>
169 <And>
170- <Not><Category>Core</Category></Not>
171- <Not><Category>Screensaver</Category></Not>
172- <Not><Category>X-GNOME-Settings-Panel</Category></Not>
173- <Not><Category>System</Category></Not>
174+ <Not>
175+ <Category>Core</Category>
176+ <Category>Screensaver</Category>
177+ <Category>X-GNOME-Settings-Panel</Category>
178+ <Filename>debian-xterm.desktop</Filename>
179+ <Filename>debian-uxterm.desktop</Filename>
180+ <Filename>ibus.desktop</Filename>
181+ <Filename>empathy-accounts.desktop</Filename>
182+ </Not>
183+ <Or>
184+ <Not><Category>Settings</Category></Not>
185+ <And>
186+ <Category>Settings</Category>
187+ <Or>
188+ <Category>DesktopSettings</Category>
189+ <Category>Security</Category>
190+ </Or>
191+ </And>
192+ </Or>
193 </And>
194 </Include>
195 </Menu> <!-- End Other -->
196
197+ <Layout>
198+ <Merge type="menus" />
199+ <Menuname>Other</Menuname>
200+ <Merge type="files" />
201+ </Layout>
202+
203 <!-- The Debian menu -->
204 <Menu>
205 <Name>Debian</Name>
206@@ -276,12 +202,4 @@
207 <Directory>Debian.directory</Directory>
208 </Menu>
209
210-
211-<!-- Separator between menus and gnome-app-install -->
212-<Layout>
213- <Merge type="menus"/>
214- <Merge type="files"/>
215- <Separator/>
216-</Layout>
217-
218 </Menu> <!-- End Applications -->

Subscribers

People subscribed via source and target branches