SasView / sasview

Code for the SasView application.
BSD 3-Clause "New" or "Revised" License
47 stars 41 forks source link

2885 logos do not appear in about box #2894

Open lucas-wilkins opened 3 weeks ago

lucas-wilkins commented 3 weeks ago

Fixes #2885 - icons not appearing

lucas-wilkins commented 2 weeks ago

One suggestion - the new buttons give no hover feedback they are links. I only knew they were because of the code.

I can fix that, probably

lucas-wilkins commented 2 weeks ago

@krzywon There were some limitations on what kind of feedback works, as image handling in Qt is pretty unpredictable and the logos are all different sizes and formats. Hope just removing the border is enough.

wpotrzebowski commented 1 week ago

Tested on Mac. It looks nice (much nicer than the previous one). Maybe 6.0.0b1 font can be slightly bigger.

The question is if we should add more logos to match updated contributors list? This will make this box probably too big and I am thinking if it is not better to skip the logos and just refer yo people list on website.

Screenshot 2024-06-25 at 06 05 54
butlerpd commented 3 days ago

check and see if we are missing any logos that really need to be added.