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

Figure out how to skin the gutter correctly #35

Open
deplorableword opened this issue Sep 3, 2012 · 7 comments
Open

Figure out how to skin the gutter correctly #35

deplorableword opened this issue Sep 3, 2012 · 7 comments

Comments

@deplorableword
Copy link
Owner

The latest build of TM2 has support for the gutter, if possible we should try and theme if accordingly textmate/textmate#233

@timhatch
Copy link

timhatch commented Oct 9, 2012

I've just taken a look at this, with some guidance from @infininight as to how gutter theming has been implemented and will submit a pull request.

@deplorableword
Copy link
Owner Author

Cool, look forward to seeing it :)

On 9 October 2012 18:21, tim [email protected] wrote:

I've just taken a look at this, with some guidance from @infininighthttps://github.com/infininightas to how gutter theming has been implemented and will submit a pull
request.


Reply to this email directly or view it on GitHubhttps://github.com//issues/35#issuecomment-9270298.

Tom Martin
twitter.com/deplorableword
devart.org

@timhatch
Copy link

Tom,

I've pushed gutter modifications for the solarized-dark and solarized-light themes to the textmate/textmate-solarized repository. Although I now see that the textmate/textmate-solarized repository is technically a fork of this one...

I got a little confused at first by all the sublime-related issues/pull requests...

@deplorableword
Copy link
Owner Author

ah I see, right I'll try to pull this upstream from textmate/textmate-solarized :) Could you paste the commit hashes in this ticket?

Yes, sublime is apparently 100% compatible with TM themes, so as long as they don't mess with TextMate I'm happy to accept them.

@timhatch
Copy link

The commit hashes are:
200b8b9
51661c7

I am aware that in principle Sublime recognizes Textmate themes and so yes, modifications to the core syntax highlighting within the theme files should work equally well in both applications. But equally the two applications appear to use the theme file for certain application-specific settings. e.g.

Textmate now has this concept of gutterSettings which so far as I know is unique to Textmate2 (so I guess will be ignored by Sublime); and

Sublime seems to have an additional editor concept of SublimeBracketHighlighter - which I hardly suppose is supported by Textmate/Textmate2

At the moment, the two editors are probably sufficiently close that these few divergent features aren't important, though I would have thought it more sensible to have a repository with the core syntax highlighting that is common between them and forks to deal with application-specific features. That would perhaps reduce the likelihood of future breakage.

But it's not my world :)

@deplorableword
Copy link
Owner Author

Yep I agree, if Sublime adds a bunch of stuff which is significantly different to TM then it should go into a fork :)

Thanks for the commit refs I'll pull those in and close when I'm done.

@timhatch
Copy link

@braver commented on the second of the above commits (51661c7) noting that it addresses an issue currently under discussion in Issue #15 of this repository and for which various people have different solutions, including pull request #39 (text selection highlighting).

I've looked at the issue more closely and replied (textmate@51661c7#commitcomment-1990784) but you may wish to ignore 51661c7 until such time as there is a consensus solution

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

No branches or pull requests

2 participants