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

Please make the icon more convenient #2

Open
chaoshuaihaohao opened this issue Jan 20, 2021 · 3 comments
Open

Please make the icon more convenient #2

chaoshuaihaohao opened this issue Jan 20, 2021 · 3 comments

Comments

@chaoshuaihaohao
Copy link

This program is very good, I use asciiflow is also very uncomfortable, and asciiflow can't export complete text, so I use it very inconvenient. But the blogger's program is also some imperfect, the icon is not very intuitive, the line is not very beautiful. I hope it can be more convenient and practical. Recently I saw the monodraw software of MacOS, it seems very good, but I use the Linux system,,, very sad, No good ASCII drawing software

@asmwarrior
Copy link

I also agree with the OP's suggestion, the icon of this project should be improved.

If you would like to see the screen shot of the main frame and the icons, you can see this discussion, I have post some screen shot images there:

This project can be built with the wxWidget library?

Generally, I think some free/open icon can be used, there are many web site to allow us to download those kinds of icons.

@asmwarrior
Copy link

New File Vector SVG Icon (10) - SVG Repo

I use the icon from the above link, and I first download the png file, which is 256*256 pixel size, than I open the GIMP, and convert this png to xpm file.

Now, here is the result application.

image

So, this is the direction to improve.

But I see when compiling, I got some gcc warning:

F:\code\AsciiFlowQT\src\Common\include/AsciiFlowCommon/ico/New.png.xpm:3:1: warning: ISO C++ forbids converting a string constant to 'char*' [-Wwrite-strings]

It looks like the xpm file has some issue? But when I export the xpm file in GIMP, I don't see any option I can tweak. Only one option is the transparent threshold.

@asmwarrior
Copy link

asmwarrior commented Jul 22, 2023

OK, I think we have to edit the xpm file, and add the const.

See discussion below: Warning in C++17 when including XPM image, though this project needs C++ 20.

asmwarrior added a commit to asmwarrior/AsciiFlowQT that referenced this issue Jul 22, 2023
See discussion here: Please make the icon more convenient · Issue schorsch1976#2 · schorsch1976/AsciiFlowQT — schorsch1976#2
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