Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Use XSETTINGS to get theme without a DE #1831

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vincentbernat
Copy link

Description

GTK is using XSETTINGS as the primary information for theme, icons and
font. It surpasses what's inside the configuration files. DE will
advertise the values they have in their own registries to XSETTINGS.
Without a DE, users can get XSETTINGS with xsettingsd. It is shipped
with dump_xsettings to get the current values.

Unrelated, but without a DE, I doubt the content of gsettings matter
much. GTK will not read it by itself (this is not related to GtkSettings).
I would remove it. People running a part of a DE (gnome-control-center
maybe?) will get XSETTINGS.

GTK is using XSETTINGS as the primary information for theme, icons and
font. It surpasses what's inside the configuration files. DE will
advertise the values they have in their own registries to XSETTINGS.
Without a DE, users can get XSETTINGS with xsettingsd. It is shipped
with `dump_xsettings` to get the current values.

Unrelated, but without a DE, I doubt the content of gsettings matter
much. GTK will not read it by itself. I would remove it. People
running a part of a DE (gnome-control-center maybe?) will get
XSETTINGS.
hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Aug 11, 2022
…et theme without a DE

Upstream PR: dylanaraps/neofetch#1831
Thanks to @vincentbernat

Co-authored-by: Vincent Bernat <[email protected]>
@hykilpikonna
Copy link

Thank you for your contribution!

This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.

HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.

Read the "Running Updated Original Neofetch" section for more info!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants