catppuccin / logseq

🦫 Soothing pastel theme for Logseq
MIT License
154 stars 4 forks source link

switching from light to dark theme using `t t` made undoable change to the accent of bold + italics accent ( using `***text***`) and the accent of header tools. #34

Closed SumirSeth closed 8 months ago

SumirSeth commented 8 months ago

I was previewing themes using the t t shortcut but it caused colors to swap, mainly the accent color of the bold+underline texts and the tool section in the headerbar. I have tried reinstalling the theme and refreshing logseq but nothing works. Please help! image

griimick commented 8 months ago

I checked the latest version of the theme. It is the same as what you have in the screenshot. It is possible that you were using an older version of the theme and it got updated or it was somehow misconfigured.

I am not sure about bold+italics behavior as no change has been made related to that in the theme recently, the toolbar might have broken in the recent changes. I can add that as an action item for this issue report.

SumirSeth commented 8 months ago

It could be that it was somehow misconfigured previously and changing the theme somehow "refreshed" it, but I feel like it does not make sense to have accent in the header icons when the home icon, sidebar icon and forward/backward arrows dont have an accent to them. It just adds inconsistency in my opinion.

While I also think that adding accent to the bold+italic text would be a great addition but that is ofc up to personal preferences of the devs.

griimick commented 8 months ago

Thank you! I agree with both the points. Will add them as action items.

SumirSeth commented 8 months ago

thank you so much!!