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

Create Cool_Reader #927

Open
wants to merge 2 commits into
base: master
from
Open

Create Cool_Reader #927

wants to merge 2 commits into from

Conversation

@Symbian9
Copy link

@Symbian9 Symbian9 commented Nov 11, 2018

Add @EXLMOTODEV's AppImage builds of Cool Reader.

@probonopd
Copy link
Member

@probonopd probonopd commented Nov 11, 2018

Thanks for submitting. Looks like it is missing a png icon. There is only an xpm icon, which is not ideal and currently fails the automated test.

@Symbian9
Copy link
Author

@Symbian9 Symbian9 commented Nov 11, 2018

There is only an xpm icon

Hm, there are not only XPM, but PNG and ICO icons too

@EXLMOTODEV, could you fix your AppImage build according this issue?

@EXL
Copy link

@EXL EXL commented Dec 17, 2018

@EXLMOTODEV, could you fix your AppImage build according this issue?

@Symbian9 yes
The new AppImage release version of Cool Reader app uses the PNG format icon:
https://github.com/EXLMOTODEV/coolreader/releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.