-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Upgrade theme to obsidian 1.0 #168
Upgrade theme to obsidian 1.0 #168
Conversation
@hellerbarde The patch by @pastordan also fixes the squished sidebar buttons, maybe you could incorporate these as well :) |
Am I testing this incorrectly? When using the css file in my themes folder, headers aren't retaining their colors and showing up as default black. |
George: are you using this code?
.theme-dark .modal .modal-content .vertical-tab-header,
.theme-dark .modal .modal-content .vertical-tab-content-container
.vertical-tab-content {
background-color: var(--background-secondary) !important;
}
If so, try changing --background-secondary to --background-primary and
see what happens.
…On Tue, Oct 25, 2022 at 10:15 AM George Gosieski ***@***.***> wrote:
Am I testing this incorrectly? When using the css file in my themes
folder, headers aren't retaining their colors and showing up as default
black.
—
Reply to this email directly, view it on GitHub
<#168 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE2C6QAH6S3XLX5NJAT6GFTWE72SHANCNFSM6AAAAAARMSW2AA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@GeorgeGo No, you are correct, that is one of the (many) as-of-yet-unresolved issues. (this is after all a "draft" pull request ;) ) I'm not the original theme developer and I'm not familiar with Obsidian theming, so unless @ceciliamay wants to jump in and push this forward, it'll be somewhat slow going. Additionally, I'll have very limited time this week. I'll also quickly mention this: while it is much appreciated to have everyone help with snippets, it would be even more useful for you to also identify the place they might need to go in the original code (inside the |
@inhji Thanks a lot for the suggestion, I'm working on incorporating this. Unfortunately, I don't remember how the side ribbon buttons looked originally, especially on hover. embarrassed |
@hellerbarde You know that Cecilia's working on an update, yes? Anyway, I believe this bit:
goes in _side-dock-ribbon.scss This goes in _tab-header.scss:
And all of this goes in _md-lists.scss, of course:
I had some other stuff for callouts, but that's really just customizing for myself. |
Oh, yeah, I'm aware. I'm just doing this for myself and why not put it out there for y'all.
|
Greatly appreciate the effort. 🙌 |
I'm also not familiar with obsidian themes, but I want to look at the Primary source and see if I can help out. It's just a great theme and I'm missing it. 😄 |
I have been looking and for the life of me cannot figure out why when using split panes, the left pane is offset higher than the right. And then oddly over time both sides will ride upwards. |
This bit drove me up a wall too.
…On Fri, Oct 28, 2022 at 2:05 PM merfed ***@***.***> wrote:
I have been looking and for the life of me cannot figure out why when
using split panes, the left pane is offset higher than the right. And then
oddly over time both rides will ride upwards.
—
Reply to this email directly, view it on GitHub
<#168 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE2C6QBNNF3S2JE56CZTHJDWFQPV5ANCNFSM6AAAAAARMSW2AA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey guys, the new design is a bit challenging with the same reason you guys have mentioned. So I'm pushing all the pull requests to see if these can help! I appreciate all the contributions and the discussion here! I'm also doing a bit of codebase rebuild because I found that some files were a bit all over the place and confusing. I want to open the theme to invite other contributors so that if they want a support a plugin for the theme they can easily do so. I will close and squash this but please feel free to keep the convo going! I'm asking because I'm also rebuilding the README, with a section to explain how to contribute. I will also be moving the theme to a different owner (Primary Theme org) so it might go missing/broken for a few days - apologies! |
Scrollbar Fix:
I think the scrollbars were just missing the |
Fix shifting split panes:
|
How is this going on? |
My fav theme by far, thanks @ceciliamay . I hope it gets 1.0 compatibility. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pushing all current PRs! thank you all for your contributions especially @hellerbarde. looking forward to working with the community more! will update everyone here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pushing all current PRs! thank you all for your contributions especially @hellerbarde. looking forward to working with the community more! will update everyone here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pushing all current PRs! thank you all for your contributions especially @hellerbarde. looking forward to working with the community more! will update everyone here.
new file: manifest.json modified: obsidian.css modified: scss/10_base/_var__dark-mode.scss modified: scss/10_base/_var__global.scss modified: scss/10_base/_var__light-mode.scss modified: scss/20_workspace/_file-explorer.scss modified: scss/20_workspace/_nav-header.scss modified: scss/20_workspace/_side-dock-ribbon.scss modified: scss/20_workspace/_status-bar.scss modified: scss/20_workspace/_tab-content.scss modified: scss/20_workspace/_tab-header.scss modified: scss/20_workspace/_title-bar.scss modified: scss/30_note/_md-aside.scss modified: scss/30_note/_md-blockquotes.scss modified: scss/30_note/_md-callouts.scss modified: scss/30_note/_md-checkboxes.scss modified: scss/30_note/_md-code.scss modified: scss/30_note/_md-embeds.scss modified: scss/30_note/_md-emphasis.scss modified: scss/30_note/_md-find-replace.scss modified: scss/30_note/_md-footnotes.scss modified: scss/30_note/_md-gutter-line.scss modified: scss/30_note/_md-headers.scss modified: scss/30_note/_md-highlighted-search.scss modified: scss/30_note/_md-lists.scss modified: scss/30_note/_md-tags.scss modified: scss/30_note/_md-yaml.scss modified: scss/30_note/_note-header.scss modified: scss/30_note/_typography.scss modified: scss/40_special-features/_alt-checkboxes.scss modified: scss/40_special-features/_custom-icons.scss modified: scss/40_special-features/_md-progress.scss modified: scss/50_ui-components/_buttons.scss modified: scss/50_ui-components/_community-themes-plugins.scss modified: scss/50_ui-components/_dropdowns.scss modified: scss/50_ui-components/_empty-page.scss modified: scss/50_ui-components/_kbd.scss modified: scss/50_ui-components/_modals.scss modified: scss/50_ui-components/_notifications.scss modified: scss/50_ui-components/_popovers.scss modified: scss/50_ui-components/_resize-handles.scss modified: scss/50_ui-components/_scrollbars.scss modified: scss/50_ui-components/_search-input.scss modified: scss/50_ui-components/_sliders.scss modified: scss/50_ui-components/_toggles.scss modified: scss/50_ui-components/_vault-launch-progress.scss modified: scss/60_mobile/_general-mobile.scss modified: scss/60_mobile/_phone-mobile.scss modified: scss/60_mobile/_tablet-mobile.scss modified: scss/70_plugins-core/_backlinks-outgoing.scss modified: scss/70_plugins-core/_graph-view.scss modified: scss/70_plugins-core/_outline.scss modified: scss/70_plugins-core/_slides.scss modified: scss/80_plugins-community/_calendar.scss modified: scss/80_plugins-community/_dataview.scss modified: scss/80_plugins-community/_excalidraw.scss modified: scss/80_plugins-community/_full-calendar.scss modified: scss/80_plugins-community/_hover-editor.scss modified: scss/80_plugins-community/_map-view.scss modified: scss/80_plugins-community/_sliding-panes.scss modified: scss/80_plugins-community/_style-settings__options.scss modified: scss/80_plugins-community/_style-settings__preferences.scss modified: scss/80_plugins-community/_style-settings__styling.scss modified: scss/80_plugins-community/_supercharged-links.scss new file: version-bump.mjs new file: versions.json
c665dd6
to
6f42895
Compare
No description provided.