Code review comment for lp:~canonical-dx-team/unity/unity.fix-646712

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

160 + if (_super_show_launcher)
(and following)… Can you please avoid by any mean to hardcode the keys?
The places can set dynamic values in their .place file for that for instance, so it won't scale. We will probably change the "t" and "w" shortcut as well. So having them in one place is better.

if you go over the model, I think you can get both the index and the shortcut.

review: Needs Fixing

« Back to merge proposal