Schrodinger-Hat / ImageGoNord

A tool that can convert your rgb images to nordtheme palette
https://ign.schrodinger-hat.it/
GNU Affero General Public License v3.0
831 stars 31 forks source link

Add Monalisa color palette #127

Closed MrPurple666 closed 11 months ago

MrPurple666 commented 1 year ago

Pull Request

Description

Monalisa It is a color palette based on the painting "Monalisa" by Leonardo da Vinci. Has been tested locally using the CLI.

Type of change

How Has This Been Tested?

Monalisa

nord

Test Configuration:

Checklist:

TheJoin95 commented 1 year ago

I like this kind of colors, is it coming directly from your idea or it already exist in other community? Of course, expect for Da Vinci eheh

TheJoin95 commented 1 year ago

If it's possible, I'll ask to add also the palette on the website, in the ImageGoWild section so anyone can convert their assets with this palette.

If you don't want or you are not available just open an issue requesting this implementation, thank you

MrPurple666 commented 1 year ago

I like this kind of colors, is it coming directly from your idea or it already exist in other community? Of course, expect for Da Vinci eheh

The 6 initial colors are purely extracted from Monalisa, I added some others colors to give the image conversion a better touch.

MrPurple666 commented 1 year ago

If it's possible, I'll ask to add also the palette on the website, in the ImageGoWild section so anyone can convert their assets with this palette.

If you don't want or you are not available just open an issue requesting this implementation, thank you

Certainly, I would be very grateful to have my collaboration there.

MrPurple666 commented 1 year ago

This week I was a little busy, but as requested, I added it to the site. Hope everything is ok, if you need any changes let me know. :)

TheJoin95 commented 1 year ago

Could you provide us with a screenshot of the updated website? We should add this on the PR template, but can you do it please? Thanks! You can update the first comment

MrPurple666 commented 1 year ago

Could you provide us with a screenshot of the updated website? We should add this on the PR template, but can you do it please? Thanks! You can update the first comment

Eh, apparently not. every time I try to open index.html it returns an empty page; i don't know if i did something wrong or i have to open it another way.

TheJoin95 commented 1 year ago

You need to run the application in dev env locally. You can refer to this little guide: https://github.com/Schrodinger-Hat/ImageGoNord/tree/master/src/ign-frontend#project-setup

You should have installed npm and nodejs. This website is a dynamic one and the index.html it should be empty if not exececuted or built correctly

MrPurple666 commented 1 year ago

Oh god, I forgot to comment; well i tried it several times and they all lead me to some npm/nodejs error. Sorry for the delay but I couldn't get it to run in any way.

TheJoin95 commented 11 months ago

Closing because no new contributions on this PR

TheJoin95 commented 3 months ago

@MrPurple666 my apologize to not bring up any kind of good documentation here.

I've created https://github.com/Schrodinger-Hat/ImageGoNord/pull/150 to implement monalisa palette. That PR will be set as a documentation PR

I send you a lot of love and my apologize <3

MrPurple666 commented 3 months ago

@MrPurple666 my apologize to not bring up any kind of good documentation here.

I've created #150 to implement monalisa palette. That PR will be set as a documentation PR

I send you a lot of love and my apologize <3

WOAH, thank you very much ❤️