Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed Doom acario theme #3539

Merged
merged 1 commit into from
Aug 25, 2022
Merged

Fixed Doom acario theme #3539

merged 1 commit into from
Aug 25, 2022

Conversation

Aethrexal
Copy link
Contributor

Changed the namespace style to fix the issue (#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.

Changed the `namespace` style to fix the issue (#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.
@Aethrexal Aethrexal changed the title Fixed Doom acario theme (#3533) Fixed Doom acario theme Aug 25, 2022
@pickfire
Copy link
Contributor

Can you please update the wiki screenshots as needed?

Copy link
Contributor

@pickfire pickfire left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@pickfire pickfire merged commit f0fb340 into helix-editor:master Aug 25, 2022
@Aethrexal
Copy link
Contributor Author

Can you please update the wiki screenshots as needed?

Will do!

@Aethrexal Aethrexal deleted the patch-1 branch August 25, 2022 15:42
AlexanderBrevig pushed a commit to AlexanderBrevig/helix that referenced this pull request Aug 29, 2022
Changed the `namespace` style to fix the issue (helix-editor#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.
thomasskk pushed a commit to thomasskk/helix that referenced this pull request Sep 9, 2022
Changed the `namespace` style to fix the issue (helix-editor#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.
jdrst pushed a commit to jdrst/helix that referenced this pull request Sep 13, 2022
Changed the `namespace` style to fix the issue (helix-editor#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.
herkhinah pushed a commit to herkhinah/helix that referenced this pull request Dec 11, 2022
Changed the `namespace` style to fix the issue (helix-editor#3533).

I also made the theme look a little closer to how it looks in Emacs, I did however opt to still have it slightly different as I found it easier to read with my port than on the original in Emacs.
I also sorted most keys (mainly from line 8 to 28) for the theme to be in alphabetical order, so it's easier to have a quick glance where they are.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants