-
Notifications
You must be signed in to change notification settings - Fork 34
Themes - add title bar class and color options to match sidebar background color for each theme #91
base: master
Are you sure you want to change the base?
Conversation
@saadq Perhaps a more friendly and universally satisfying approach would be to allow a feature like this as a Theme option: |
Firstly, awesome work. Didn't know about this feature, it looks great! Just wanted to confirm – were there no issues in Windows/Linux with having this extra This is just imo, but I think that the colored title bar fits the theme of Materialize pretty well and I think it should be on by default. Although I agree that there should still be a way to turn it off, so a What are your thoughts on it? |
Since Windows/Linux does not support this feature, they will simply ignore the I like your idea of having a Theme option for this which defaults to true. I'm not sure how to wire that up, but I'll have a look around at some code - I'm sure I'll be able to figure it out. Thanks for the feedback. I'll be in touch soon. |
I discovered a bit in my research for this today: For each theme, I set I also set the If you are unhappy with the addition of the extra Preferences.sublime-settings file in the root directory, I can remove it. Each user would then just have to turn it on individually in their User preferences. If you have any ideas of a better way to go about enabling this by default, let me know. I'm open to any additional feedback or insight you have. If you are happy so far, I can go ahead and add |
@saadq Another thought - I currently have the foreground (text) of the title bar set to either Black for light themes or White for dark themes. If you'd rather - and I think this would look a bit sharper - I could set the text color of the title bar to match, say, the sidebar label text color. That would be more in keeping with the overall theme as well. I'll go ahead and do that and add some screenshots to get your feedback. |
@saadq updated title_bar fg (text) colors to match theme colors. Takes from class: sidebar_label parent attribute: expandable. Line ~586 in most themes. Let me know how you feel about this and what other work I can do to push this forward. Thanks. |
Thanks very much for your work! I'm going to review this in-depth tomorrow, but it's looking really good :) |
Alright, so I just went over this. Everything looks really good – the one thing I'd like to see changed is having the title bar's color be different than the background color. Right now, I believe you have the title bar's background color to be set to the background color of the For example, with Material Spacegray you can have this at the top: {
"class": "title_bar",
"settings": ["material_theme_title_bar"],
"platforms": ["osx"],
"fg": [175, 189, 196],
"bg": [43, 48, 59]
} but then in the contrast mode options (in line 3355 in {
"class": "title_bar",
"settings": ["material_theme_contrast_mode"],
"platforms": ["osx"],
"fg": [175, 189, 196],
"bg": [41, 45, 55]
} Does that make sense? Let me know if I can clarify. |
@saadq That makes good sense. I'll work on those changes. |
Ah woops I meant |
Super excited for this change. The colored titlebar is one of my favorite features of themes |
Sorry I haven't completed this yet - I've been traveling for work more than usual lately. I'll complete these changes by this weekend. |
Sure thing, take your time and let me know if I can help at all or if you have any questions. Great work so far, looking forward to see this in Materialize :) |
Looking forward to this too! Any news on getting it merged? |
Hey @rudedogg I'm ready to merge whenever it is finished. Currently, this PR makes the title bar the color of the contrast mode. I requested changes to make the title bar color the same color as the theme, but use the contrast mode color when I imagine @paulhirschi has been busy, but I can always accept a PR from anyone else who would want to implement this if he lets me know that he doesn't have time for it. |
@saadq @paulhirschi Awesome! Happy to hear you're gonna pick it back up. If something comes up let me know, I'll give it a go! |
Hey @paulhirschi, speaking for all those that found this via google - you think you'd have time to implement the recommended changes anytime soon? |
Since Sublime Text dev build 3127 the option to change the background and foreground color of the window titlebar has been available (only available on OS X 10.10+). In this PR, I've set the titlebar bg color to match the bg color of the sidebar for each theme. See screenshots below for examples.