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

[regression] Problem with some gtk2 assets (combobox, entries) #523

Open
actionless opened this issue Aug 24, 2020 · 12 comments
Open

[regression] Problem with some gtk2 assets (combobox, entries) #523

actionless opened this issue Aug 24, 2020 · 12 comments

Comments

@actionless
Copy link
Collaborator

Reporter Info

  • Materia version: v20200320-83-g6f7e51a9
  • GTK2 version: 2.24.32-2
  • Distribution (and version): git
  • Desktop environment (and version): none
  • Related application (and version): awf-gtk2

Actual Issue
2020-08-24--1598260595_1021x1002_scrot

Steps to Reproduce
run render-assets.sh and install.sh

@afraidofmusic
Copy link

I had the same problem before but e766f45 fixed it. It might cause a regression for you if your Inkscape is old. What is the version of your Inkscape?

@actionless
Copy link
Collaborator Author

in that case it would be some image generation error in the log and also other assets generated correctly, so i think it might me some issue with elements themselves in assets.svg

@nana-4
Copy link
Owner

nana-4 commented Sep 17, 2020

So sorry for the late reply on this.

I also can't reproduce this with Inkscape 1.0 (4035a4fb49, 2020-05-01) on Fedora 32.

What version of Inkscape are you using?

I tried validating src/gtk-2.0/assets.svg with https://validator.w3.org, but didn't find errors that could be related to this issue.

@nana-4
Copy link
Owner

nana-4 commented Sep 17, 2020

This could be related to the HiDPI option, I guess?

Looking at your screenshot, some assets are 200% scale, but some others are 100% scale, apparently.

@actionless
Copy link
Collaborator Author

actionless commented Sep 17, 2020

no, it's not related to hidpi

thanks for the hint -- i've double-checked with inkscape -- so apparently the issue somehow related to resvg, looking further into it

@nana-4
Copy link
Owner

nana-4 commented Sep 17, 2020

Looking at your screenshot, some assets are 200% scale, but some others are 100% scale, apparently.

Oh, no, please ignore this. I didn't carefully check the screenshot, and all the assets seems 100% scale. Sorry 😓

@actionless
Copy link
Collaborator Author

so i've checked the problem is reproducing with resvg 0.5, 0.6, 0.8, 0.9 and 0.11 (and first two were known to work fine before)

so it must be smth changed in gtk2 assets file itself, mb added some new svg property which never was supported (or always was broken) in resvg

@actionless
Copy link
Collaborator Author

actionless commented Sep 17, 2020

unfortunately i can't check the older releases of materia as they're not compiling due to sass standard breakage in sass minor release:

2020-09-17--1600363335_1920x1080_scrot

@actionless
Copy link
Collaborator Author

and with hidpi it looks the same, only combobox and entry looks broken:

2020-09-17--1600364259_1920x1080_scrot

@actionless actionless changed the title [regression] Problem with some gtk2 assets (buttons, checkbuttons) [regression] Problem with some gtk2 assets (combobox, entries) Sep 17, 2020
@actionless
Copy link
Collaborator Author

so i suspect this commit but can't test it due to the sass build problem above: ff16cb5#diff-96d139dfb99fcb8d87ff483d9d92991c

@actionless
Copy link
Collaborator Author

actionless commented Sep 17, 2020

let's try to analyze the difference between the same asset (https://github.com/nana-4/materia-theme/blob/master/src/gtk-2.0/assets.svg) rendered by inkscape vs resvg, cc @RazrFalcon

id=combo-left-entry-active

inkscape (20x20px)
combo-left-entry-active

resvg (38x40px), asset itself rendered in 20x20, but have extra whitespace at the right-bottom
combo-left-entry-active

@RazrFalcon
Copy link

Interesting. Inkscape provides the size after clipping, while resvg returns the actual size. Not sure which one is the correct one. Especially since I don't understand how exactly Inkscape does this. It's fine for rectangular clipping, but a complex one is way more tricky. Will test it later.

In the end, resvg produces an expected result here. It's not a bug. At least at a first glance.

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

4 participants