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

feat: erc theming #147

Merged
merged 1 commit into from
Jun 4, 2024
Merged

feat: erc theming #147

merged 1 commit into from
Jun 4, 2024

Conversation

LionyxML
Copy link
Contributor

@LionyxML LionyxML commented May 7, 2024

Some github boo boo happened, I deleted my local repo and the pull request got messed, so "reopening" I guess.

Original from:
#144

Last message to @NamesCode

Hey @NamesCode! I made a mistake!

I have an extention to provide colorization, I turned it off to take the screenshots but I uploaded the wrong files, lol (https://github.com/leathekd/erc-hl-nicks by the way).

ERC by default does not make distintions between nicks in terms of colors.

Allow me some right screenshots here:

Private chat:
image

Channel chat:
image

Server buffer:
image

Regarding theming the `erc-hl-nicks' package, I would wait until the new version of ERC comes to Emacs 29. It should change some behaviours, how knows if we get some new faces :)

@LionyxML LionyxML changed the title Feat/erc theming feat: erc theming May 7, 2024
catppuccin-theme.el Outdated Show resolved Hide resolved
catppuccin-theme.el Outdated Show resolved Hide resolved
catppuccin-theme.el Outdated Show resolved Hide resolved
catppuccin-theme.el Outdated Show resolved Hide resolved
catppuccin-theme.el Show resolved Hide resolved
Copy link
Member

@jtbx jtbx 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 thanks. It would be awesome if you could squash the 5 commits into one to remove the rubbish github added after that review. 😄

@LionyxML
Copy link
Contributor Author

LionyxML commented May 9, 2024

Thanks @jtbx!

The interface is new for me, I found I could do it on the last 3, lol.

I'll try to fix it, but idn, I wish a "squash commits" option here.

Co-Authored-By: Jeremy Baxter <jtbx@duck.com>
@LionyxML
Copy link
Contributor Author

LionyxML commented May 9, 2024

@jtbx looks like "I did it" :)

@jtbx
Copy link
Member

jtbx commented May 9, 2024

Awesome thanks a lot. :)

@NamesCode
Copy link
Member

Sorry for being so late to this. It looks amazing, thanks ^^

@NamesCode NamesCode merged commit 729e6be into catppuccin:main Jun 4, 2024
1 check passed
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.

3 participants