efemkay / obsidian-modular-css-layout

CSS Layout hack for Obsidian.md
https://efemkay.github.io/obsidian-modular-css-layout/
GNU General Public License v3.0
918 stars 74 forks source link

Multi-column broken with Minimal Theme #49

Closed Rikiub closed 1 year ago

Rikiub commented 1 year ago

This is a multi-column by 3x3

Default Theme (Expected)

Default Theme

Minimal Theme (Bug?)

Minimal Theme

Not only does the layout change, the image is also "cropped" on the right.

All the themes I have tested act like the "Default Theme", it only happens with the Minimal Theme.

efemkay commented 1 year ago

hey @Rikiub can you share the raw markdown? i'm not sure what u mean by "multi-column by 3x3". don't have specific 3x3 setups. so far i've only 3 columns on single row or multi column multi row that will be responsive to your screensize (the smaller screen will have less columns)

Rikiub commented 1 year ago

Ah sorry, I was not very precise.

This is the Markdown:

> [!multi-column]
> 
>> [!blank]
>> ![[image.jpg]]
>
>> [!blank]
>> ![[image.jpg]]
>
>> [!blank]
>> ![[image.jpg]]
>
>> [!blank]
>> ![[image.jpg]]
>
>> [!blank]
>> ![[image.jpg]]
efemkay commented 1 year ago

the number of columns is as per mentioned. it's dynamic based on sub-callout width setting (can change via style settings). the picture "cut off" is the same issue as per #50 . will work on the fix

i'm gonna close this issue (as the number of columns is as per design). this won't work right now because of the issue #50 but once it's fixed, you should use gallery callout instead if you want fixed number of columns for picture. for multi-column, i am working on a more robust implementation of columns options. you'll have to wait a bit for that