PaulCombal / SamRewritten

Steam Achievement Manager For Linux. Rewritten in C++.
GNU General Public License v3.0
334 stars 32 forks source link

GUI Error and rename project #75

Closed rejexy closed 4 years ago

rejexy commented 4 years ago

Latest version of SAM has problem with GUI. image

What do you think about changing the name of the project? In general I'm used to "rewritten", but at this stage the project may have its name like, NeoSAM (neo vim), SAMium (chromium, codium) or like GOG, SAM Galaxy, in the end just SAM.

PaulCombal commented 4 years ago

Hmm that's a weird issue.. are you sure it's not a problem with your settings? Can you tell me what distro and desktop you're using? As of the project name, maybe I'll change it but believe it or not I still have plenty to do before thinking of that seriously haha. Nice name ideas though!

rejexy commented 4 years ago

Arch Linux. Gnome-shell. I installed the package over of the old one, maybe this is the problem.

rejexy commented 4 years ago

Well, i tested AppImage, same error.

image

Also, the menu looks too small.

image

wgpierce commented 4 years ago

I don't see any reason for renaming it. It's not a "new, better" SAM, it's a port/rewrite to Linux.

I think you're missing 🏅 U+1F3C5 emoji in your emoji pack, so it just displays a big blank box. Your "extra info" icons are also broken; master has a fix for that.

rejexy commented 4 years ago

I don't see any reason for renaming it. It's not a "new, better" SAM, it's a port/rewrite to Linux.

I think you're missing U+1F3C5 emoji in your emoji pack, so it just displays a big blank box. Your "extra info" icons are also broken; master has a fix for that.

Yes, I think to replace with 🏆 U+1F3C6. As for the name, regardless of whether it is rewritten or not, it seems to me that it is not correct to indicate this in the name. Better to transfer to the description.

wgpierce commented 4 years ago

Yes, I think to replace with trophy U+1F3C6.

If that works, then your emoji pack is broken.

As for the name, regardless of whether it is rewritten or not, it seems to me that it is not correct to indicate this in the name. Better to transfer to the description.

I don't see a problem with it. Especially for something like this which is emphatically not the original SAM.

rejexy commented 4 years ago

If someone endorse the name change, we can create a new issue for discussion. I like NeoSAM.

wgpierce commented 4 years ago

Original bug has been fixed. Maybe Paul feels differently, but I don't have a strong desire to rename at this time, so closing this issue.