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

Refactor solarized-definitions.el for customization #191

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sambrightman
Copy link

@sambrightman sambrightman commented Oct 20, 2016

  • add macro solarized-with-color-variables
  • pull solarized-apply-definitions out of create-solarized-theme

This makes it easier to customize and test proposed changes to
solarized-color-definitions. solarized-with-color-variables allows
definition of faces within the same context that is used for
solarized-color-definitions. Hence, the changes you test yourself can
be directly copied into the official code-base for a PR. They are also
easy to maintain in your own Emacs init scripts before merge or if
tweaking the official settings. The approach is copied from @bbatsov's
themes.

For example, setting up additions/customizations goes like this:

(defvar my/solarized-extra-definitions
  (solarized-with-color-variables
    `(;; cperl
      (cperl-array-face (,@fg-blue))
      (cperl-hash-face (,@fg-blue))
      (cperl-nonoverridable-face (,@fg-magenta)))))
(solarized-apply-definitions my/solarized-extra-definitions 'solarized)

* add macro solarized-with-color-variables
* pull solarized-apply-definitions out of create-solarized-theme

This makes it easier to customize and test proposed changes to
solarized-color-definitions. solarized-with-color-variables allows
definition of faces within the same context that is used for
solarized-color-definitions. Hence, the changes you test yourself can
be directly copied into the official code-base for a PR. They are also
easy to maintain in your own Emacs init scripts before merge or if
tweaking the official settings. The approach is copied from @bbatsov's
themes.

For example, setting up additions/customizations goes like this:

(defvar my/solarized-extra-definitions
  (solarized-with-color-variables
    `(;; cperl
      (cperl-array-face (,@fg-blue))
      (cperl-hash-face (,@fg-blue))
      (cperl-nonoverridable-face (,@fg-magenta)))))
(solarized-apply-definitions my/solarized-extra-definitions 'solarized)
@sellout
Copy link
Owner

sellout commented Feb 10, 2023

I really like this and would love to integrate it. However, the code base has moved around a bit, and it would need to be updated to be merged. I’m happy to take it across the finish line, but it’s your work, so I want to leave it to you, if you’re still interested after all this time. Let me know. Thanks!

@sambrightman
Copy link
Author

I still prefer this package to the more popular ones, to the extent I'm copying it around all my installations after it was removed from MELPA. Will it be making a comeback? If so, I'd love to get this integrated (and you're welcome to do it yourself if I don't find time).

@sellout
Copy link
Owner

sellout commented Mar 18, 2023

That's nice to hear. I have a bunch of updates that have accumulated and should have them all up shortly, then yes, it should be back on MELPA soon. It will also have a Nix flake, if that's your style.

I might be able to get this integrated tonight.

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.

None yet

2 participants