b4n left a comment (geany/geany#3614)

I just rebased and updated to try and install a fallback icon instead of the 
questionable change.  I also dropped the hack not to introduce a new string, 
which is irrelevant now.
With the last change, we could get rid of most of the other changes, but in 
some instances using the symbolic icon seems reasonable, so we could do it 
anyway.  But we could also be conservative and just include the missing icons 
and call it a day.

Opinions?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3614#issuecomment-2864163849
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3614/[email protected]>

Reply via email to