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

[REQUEST] Add a fix for icons path for main line Rofi. #132

Closed
2 tasks done
musicfunfan opened this issue May 6, 2024 · 5 comments
Closed
2 tasks done

[REQUEST] Add a fix for icons path for main line Rofi. #132

musicfunfan opened this issue May 6, 2024 · 5 comments

Comments

@musicfunfan
Copy link

musicfunfan commented May 6, 2024

Before opening a feature request

  • I checked the next branch to see if the feature has already been implemented
  • I searched existing reports to see if it is already requested.

What is the user problem or growth opportunity you want to see solved?

In the main line Rofi there is a fix. I want that added to this fork. In order to have icons on the applications that do not have .png and the .svg extensions.

How do you know that this problem exists today? Why is this important?

I know this problem exist. Because i use the AM package manager for appimages. And the icons do not displayed on the Rofi. I did the research and find this issue.

Who will benefit from it?

I think all the people use packages that the icons do not have the file extensions will be benefit from this change.

Rofi version (rofi -v)

Version: 1.7.5+wayland2-1-gff2338c3

Configuration

My config files is here.

Additional information

I use archlinux. BTW

@lbonn
Copy link
Owner

lbonn commented May 6, 2024

@musicfunfan the fix commit you are referring to is present in the wayland branch (I've merged upstream changes a couple of days ago).

Maybe you can try it on a fresh build?

@musicfunfan
Copy link
Author

I do not see an update on the AUR.

yay -Sua
:: Searching AUR for updates...
 -> Packages not in AUR: stacer
 there is nothing to do

Is it updated on A.U.R. ?

What do you mean with term "fresh build" ?

@musicfunfan
Copy link
Author

So... i re-install the package this time form archrepo i did not know that this package was on the arch repo... But i have the same problem... Is there anything more i can try ? Did i do something wrong here?

If anyone knows something about this feel free to mention me. Thanks in advance. 😉

@lbonn
Copy link
Owner

lbonn commented May 13, 2024

@musicfunfan this package contains the latest released version, without this fix. If you really need a fix from the latest version on git and you are using AUR, this is probably what you are looking for: https://aur.archlinux.org/packages/rofi-lbonn-wayland-git

@musicfunfan
Copy link
Author

Ok, yea i use A.U.R. This fix will included on the latest release ? If yes then i am happy to wait, because its not a big problem and i try when its possible not to use the A.U.R. when an official package exist.
Just let me know if the A.U.R. (git package) will be the official package some day.

Anyway, thanks for the answer and your time.

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

No branches or pull requests

2 participants