regolith-linux / regolith-styles

A convention for organizing Xresource-based theme data.
GNU General Public License v3.0
56 stars 21 forks source link

Some apps do not following the theme #14

Closed cheginit closed 4 years ago

cheginit commented 4 years ago

Some apps like GIMP and slack don't follow the theme button and window style. For example, this is what GIMP looks like. Notice the ancient-looking of the interface!

Screenshot from 2020-02-18 10-35-31

Firefox doesn't have this issue. For example, this is how the open dialog box looks like: Screenshot from 2020-02-18 10-36-12

kgilmer commented 4 years ago

Were these apps installed via snaps? I believe there are custom flags required to all snap-based apps to get access to the local theme resources...

cheginit commented 4 years ago

No, I installed GIMP from its PPA. I installed slack from snap though and it has an issue with the tray icon. I looked at the syslog and it says that icon cannot be found for snap:

Mar  5 00:14:48 titan gnome-software[3579]: g_path_get_basename: assertion 'file_name != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: g_regex_match_full: assertion 'string != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: g_path_get_basename: assertion 'file_name != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: g_regex_match_full: assertion 'string != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: Failed to load snap icon: local snap has no icon
Mar  5 00:14:48 titan gnome-software[3579]: failed to call gs_plugin_refine_app on snap: Invalid string value converting to GVariant
Mar  5 00:14:48 titan gnome-software[3579]: g_variant_new_variant: assertion 'value != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: g_variant_new_variant: assertion 'value != NULL' failed
Mar  5 00:14:48 titan gnome-software[3579]: failed to call gs_plugin_refine_app on snap: Invalid string value converting to GVariant
cheginit commented 4 years ago

The issue with the snap icon is because of snap. So I found a way of automatically installing the latest version from a deb file and uninstalled the snap version.

version=$(curl -s "https://slack.com/release-notes/linux" | awk -v FS="(<h2>|</h2>)" '{print $2}' | grep -m1  Slack | awk -F' ' '{ print $2}') && \
wget https://downloads.slack-edge.com/linux_releases/slack-desktop-${version}-amd64.deb
kgilmer commented 4 years ago

Are we good here @cheginit ?

cheginit commented 4 years ago

Yeah, I think nothing can be done from Regolith side.