appDisplay: Remove Frequent view
The Frequent apps grid has a few problems: * On a fresh install there would be no history of app usage so the applications shown in the grid have no relevance it takes time to be useful instead of being useful from the start; * The grid has far too many items in it to be relevant; 24 apps is well beyond the average use case as most people don't frequently use that many, so it gets populated with several apps that are single use (hello xterm); * The position of items in the grid are always changing based on an unknown frequency metric (and not by user-intended input) which makes it a poor way to quickly launch apps as one would have to constantly learn the positions of the items in the grid; * Having two app grids is a bit superfluous and needlessly complicates the app launching navigation: you have to spend time checking the frequent grid and if it's not there you have to switch over to another grid and find the app you need in there it's not straightforward. Remove the Frequent tab and simplify the related code. Related: https://gitlab.gnome.org/GNOME/gnome-shell/issues/1425 https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/880
This commit is contained in:

committed by
verdre

parent
4ae04d5aa8
commit
548e19a7cd
@ -14,8 +14,6 @@ $app_icon_padding: 24px;
|
||||
}
|
||||
}
|
||||
|
||||
//.app-display { spacing: 20px; }
|
||||
|
||||
/* App Icons */
|
||||
|
||||
$app_grid_fg_color: #fff;
|
||||
@ -123,15 +121,11 @@ $app_grid_fg_color: #fff;
|
||||
}
|
||||
|
||||
// Some hacks I don't even know
|
||||
.all-apps,
|
||||
.frequent-apps > StBoxLayout {
|
||||
.all-apps {
|
||||
// horizontal padding to make sure scrollbars or dash don't overlap content
|
||||
padding: 0px 88px 10px 88px;
|
||||
}
|
||||
|
||||
// Label when no frequent apps
|
||||
.no-frequent-applications-label { @extend %status_text; }
|
||||
|
||||
// shutdown and other actions in the grid
|
||||
.system-action-icon {
|
||||
background-color: rgba(0,0,0,0.8);
|
||||
@ -139,44 +133,3 @@ $app_grid_fg_color: #fff;
|
||||
border-radius: 99px;
|
||||
icon-size: $app_icon_size * 0.5;
|
||||
}
|
||||
|
||||
/* Frequent | All toggle */
|
||||
|
||||
// container
|
||||
.app-view-controls {
|
||||
padding-bottom: 32px;
|
||||
}
|
||||
|
||||
// buttons
|
||||
.app-view-control {
|
||||
padding: 4px 32px;
|
||||
margin: 0 4px;
|
||||
|
||||
&, &:hover, &:checked {
|
||||
@include button(undecorated);
|
||||
color: darken($osd_fg_color, 25%);
|
||||
}
|
||||
|
||||
&:hover {
|
||||
color: $osd_fg_color;
|
||||
box-shadow: inset 0 -2px darken($osd_fg_color, 25%);
|
||||
}
|
||||
|
||||
&:active {
|
||||
box-shadow: inset 0 -2px $osd_fg_color;
|
||||
}
|
||||
|
||||
&:checked {
|
||||
color: $osd_fg_color;
|
||||
box-shadow: inset 0 -2px $selected_bg_color;
|
||||
}
|
||||
|
||||
&:first-child {
|
||||
border-right-width: 0;
|
||||
border-radius: 0;
|
||||
}
|
||||
|
||||
&:last-child {
|
||||
border-radius: 0;
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user