Prism-Theme icon indicating copy to clipboard operation
Prism-Theme copied to clipboard

support "alternate checkboxes"

Open eleanorkonik opened this issue 3 years ago • 3 comments

It would be helpful if you supported some of the data-task checkboxes that are becoming more common in the community, i.e. this snippet.

https://publish.obsidian.md/hub/02+-+Community+Expansions/02.05+All+Community+Expansions/CSS+Snippets/Alternate+Checkboxes+(SlRvb)

Right now they're rendering like this:

image

Which makes it hard for me to use your (very beautiful and clear!) theme

eleanorkonik avatar Jan 01 '22 21:01 eleanorkonik

Hi @eleanorkonik, I'm finally coming around to this issue. I've had a look at SlRvb's Checkboxes snippet and it appears to work fine with the theme as you can see below. Unless you meant for Prism to support it in a more native way (integrated into the theme) with a more matching style to my theme? I'd be happy to take a crack at it if that's the case. There are definitely some colours that would need adjusting for best readability.

Light Theme Dark Theme
image image

damiankorcz avatar Jan 25 '22 16:01 damiankorcz

I was mostly concerned with it not showing up in Live Preview's "editing" mode, not so much the color optimizations and stuff. If they work now, awesome!

eleanorkonik avatar Jan 25 '22 19:01 eleanorkonik

it will be nice if Prism-theme supports this snippet natively @damiankorcz

AiurArtanis avatar Jan 29 '22 06:01 AiurArtanis

Hi @eleanorkonik and @AiurArtanis, After a lot of deliberation about this feature, I've chosen not to implement it natively in Prism. Main point of contention for me has been the fact that despite Alternative Checkboxes appearing across multiple popular themes, there is no standardisation between the syntax used and all of them do their own take on it. This would make supporting the feature difficult since I would likely have to support all the different variants for people coming from those themes. I also don't want to do my own take on it since that would only add to fragmenting things for users.

The latest Prism release (3.0.0) should work and look fine with the latest ITS Checkboxes snippet: https://github.com/SlRvb/Obsidian--ITS-Theme/blob/main/Snippets/S%20-%20Checkboxes.css

If there are other iterations on this feature in a CSS Snippet form and they don't work right in Prism, please let me know and I'll see what I can do to make sure it does.

Apologise for keeping this issue open so long. It's been on my mind this whole time and I didn't know how to go about it 😅

damiankorcz avatar Nov 12 '22 18:11 damiankorcz