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
Drawpile Icon #2 #3310
Drawpile Icon #2 #3310
Conversation
On Wed, Oct 31, 2018 at 10:06:00AM -0700, fbrosson wrote:
@humdingerb, @Begasus, what header would you recommend here?
* `resource(1) #'VICN' array {`
* `resource #'VICN' array {`
* `resource vector_icon array {`
* `resource vector_icon {`
They all do the same thing. It seems convenient to have the app icon
always stored with the id 1, so I'd keep that. In case a file has
multiple icons stored as resources, it makes it easy to find the icon
to use for the app.
|
@pulkomandy so do you think I should keep |
On Wed, Oct 31, 2018 at 07:19:42PM +0000, Bach Nguyen wrote:
@pulkomandy so do you think I should keep `resource(1) #'VICN' array {`?
Yes that's fine.
|
We currently have:
If all these strings are equivalent (except for the index), why not use what is already the biggest trend? @bach5000, I'd keep the |
@humdingerb @fbrosson GCI website doesn't work. I'm trying submit the task for proposing Haiku's mascot but I can't. |
@bach5000, I just left a message in the task instance, 5894692148346880. |
I recently uploaded a file with QupZilla, via drag&drop even... |
Is this PR ready to be merged or are there any outstanding issues? |
@bach5000 can you recheck this one and let us know the status, it'd be nice to get it merged soon. |
@scottmc I believe it's ready to be merged. |
@fbrosson @humdingerb @Begasus This is the new PR for my icon :)
Should fix #3261 If everything was done correctly.