Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Theme selector creates a long list of entries in config.rasi whenever new theme was selected via Rofi Theme Selector #167

Open
2 tasks done
JaKooLit opened this issue Feb 7, 2025 · 2 comments · May be fixed by #168
Labels

Comments

@JaKooLit
Copy link

JaKooLit commented Feb 7, 2025

Rofi version (rofi -v or git commit in case of build issue)

Version: 1.7.8+wayland1-dirty

Configuration

https://gist.github.com/JaKooLit/4637ceef81e81904419becce74e705b3

Theme

https://gist.github.com/JaKooLit/94aaaa9a5826d53dcc1c891bf375ad65

Timing report

https://gist.github.com/JaKooLit/c0b90e239eb7e45ae9cd06abba1047bc

Launch command

launch Rofi Theme Selector

Step to reproduce

launch Rofi Theme Selector and then select a new theme

check contents of config.rasi

Image

Expected behavior

instead of writing a new line,

to just overwrite the line

@theme

Actual behavior

creating a long list of @themes

Additional information

I dont know what to say more

Using wayland display server protocol

  • Yes, I use rofi with wayland support

I've checked if the issue exists in the latest stable release

  • Yes, I have checked the problem exists in the latest stable version
@lbonn
Copy link
Owner

lbonn commented Feb 7, 2025

@JaKooLit the actual reproduction is to change theme with rofi-theme-selector several times in a row (eg: with several command invocations). Is that right?

In that case I don't really see a problem with the current version, probably it's even a feature:

  • it's functional, the latest selected theme will be applied
  • user will have an history of the themes they've tried in the config file
  • when they've made up their mind, they can manually clean up the config file which is plain text and is meant to be edited "by hand" anyway

rofi-theme-selector is not really meant to be used every day, it's more of a one-time helper afaik.

If you want to switch between themes on a regular basis, it probably calls for several config files.

@JaKooLit
Copy link
Author

JaKooLit commented Feb 7, 2025

@lbonn

Thanks...

Could be a feature but some users may keep changing and at some point, there will be a long list. if users forgot to tinker or check

Perhaps Ill change my PR then to limit the history to say 10? or 20?

Its true, its not designed to change themes everyday but there are some handful users who wants to keep try other themes.

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

Successfully merging a pull request may close this issue.

2 participants