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

Add Qt theme detection + Option #1713

Closed
wants to merge 2 commits into from
Closed

Add Qt theme detection + Option #1713

wants to merge 2 commits into from

Conversation

capezotte
Copy link

Detects Qt widget style in LXQt, Trinity and, KDE Plasma, as well as GTK environments with Qt5ct. If they are the same as their GTK counterparts, display them as one entry.

Some caveats:

  • I replaced the kde_theme environment variable with a togglable qt_theme environment variable. Should be fairly easy to add back in, though, if there is demand.
  • Qt5ct stores fonts in a binary format. If portability is not an issue, the implementation contained in this pull request should be replaced with one using od --endian=big.

Comment on lines +2053 to +2055
LXDE*) ob_file="lxde-rc" ;;
LXQt*) ob_file="lxqt-rc" ;;
*) ob_file="rc" ;;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was fine as-is.

hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Aug 11, 2022
… + Option

Upstream PR: dylanaraps/neofetch#1713
Thanks to @capezotte

Co-authored-by: Carlos Eduardo <carana2099@gmail.com>
@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!

@hykilpikonna
Copy link

Also just realized that this would set theme to ] when no themes are present...
image
image

hykilpikonna added a commit to hykilpikonna/hyfetch that referenced this pull request Aug 14, 2022
@hykilpikonna
Copy link

Fixed

image

@capezotte capezotte closed this Jan 23, 2023
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.

4 participants