Skip to content

added theme config and screenshot image for dark-lue theme :) - #66

Open
collmut wants to merge 2 commits into
geany:masterfrom
collmut:dark-lue-theme
Open

added theme config and screenshot image for dark-lue theme :)#66
collmut wants to merge 2 commits into
geany:masterfrom
collmut:dark-lue-theme

Conversation

@collmut

@collmut collmut commented Nov 20, 2023

Copy link
Copy Markdown
Contributor

Added new theme.
I was to make index but can't at the moment because of few dependencies that i need to install before doing that.
I like Geany editor alot. Regards!

@eht16 eht16 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @collmut,
sorry for the late reply, I missed to notice the PR.

No worries about generating the index, I'll do after merging.

Could you add yourself to the AUTHORS file in the repository?

# incremented automatically, do not change manually
version=
author=Collins Mutugi
url=collinsey.itch.io

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
url=collinsey.itch.io
url=https://collinsey.itch.io

name=dark-lue
description=An easy dark & blue theme.
# incremented automatically, do not change manually
version=

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you set a version here, just 1.0 if you don't mind.

#

[theme_info]
name=dark-lue

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you mind to change the name to "Dark-Lue" so it fits better with the other themes' naming?

@eht16

eht16 commented Nov 9, 2024

Copy link
Copy Markdown
Member

Geany warns about an unknown named style:
Unknown named style '0x484848'

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

Successfully merging this pull request may close these issues.

2 participants