riscv-non-isa/riscv-iommu

Fix note icon, UL marker size, dot-leader

Closed this issue · 37 comments

wmat commented

Revert the note admonition icon to the older info circle.
Make unordered lists bullets larger.
Add the TOC dot-leader back in.

Change codespan font to Jetbrains mono.

Make page number column in TOC fixed width so the column does not look wavy.
Make Figure number and Table number column in list of figures and list of tables fixed width so that captions line up and dont look wavy. Perhaps something like below:

# List of figures settings
lof:
  label_width: '3em'  # Adjust the width for figure labels
  align: left
  caption:
    indent: '4em'  # Ensure captions align consistently

# List of tables settings
lot:
  label_width: '3em'  # Adjust the width for table labels
  align: left
  caption:
    indent: '4em'  # Ensure captions align consistently

Bullets could possible be fixed this way:

# List settings
list:
  indent: '$horizontal_rhythm * 1.5'
  item_spacing: '$vertical_rhythm / 2'
  bullet:
    font_size: 'round($base_font_size * 1.5)'
    content: '•'
wmat commented

I'll give that a try.

wmat commented

Didn't seem to work.

wmat commented

I've fixed the bullets. I have it looking like this now:

Screenshot from 2024-06-10 10-17-51

That looks good. Thanks!

how does the level 2 and level 3 sub-bullet look?

wmat commented

Here's what all 3 levels look like:

Screenshot from 2024-06-10 11-03-48

The size and shape look good! Could the second level also have color black instead of white.

wmat commented

How's this:
Screenshot from 2024-06-10 11-25-48

Thanks! Looks good!

wmat commented

btw, setting the :bibtex-style: apa, which matches the ISA manuals will fix your bibliography alignment issue.

I think the LaTex ISA manual - before asciidoc coversion - used to use ieee bibtext style. It would be preferred to use ieee style..
image

wmat commented

Ah, OK. This is a known issue with the ieee style in asciidoc. I'll see if there's a resolution.

@wmat - would you be updating the codespan font as well to something like Jetbrains mono?

wmat commented

Yes, I'll do that as well.

wmat commented

JetBrainsMono has been added to my docs-resources branch and pointed codespan at it.

As for the Bibliography, it'd be some work to make the ieee formatting match LaTeX. I know this isn't ideal, but I don't think it's warranted right now. Can you live with the layout the way it is?

So what's left for the iommu spec? The alignment of the LoF and LoT?

Thanks!

As for the Bibliography, it'd be some work to make the ieee formatting match LaTeX. I know this isn't ideal, but I don't think it's warranted right now. Can you live with the layout the way it is?

I think I can live with that.

So what's left for the iommu spec? The alignment of the LoF and LoT?

The two other item I was tracking were

  1. "wavy" numbers column in TOC, LoF, and LoT.
  2. Lining up the caption text to start from same column in LoF and LoT.
wmat commented

So if I use JetBrainsMono as the font for lists it's slightly approved, at least for text alignment but I can't seem to get the wavy numbering to go away:

Screenshot from 2024-06-11 13-48-50

The Jetbrains mono was to help solve the weird ligatures that were being previously produced like this "&". Hopefully updating to Jetbrains fixed that

image

wmat commented

Looks like this now:
Screenshot from 2024-06-11 14-04-31

Thanks! This looks nice! The older one rendered "&" and "~" using really odd characters.

wmat commented

OK, great. That screen capture was from my local build btw. I'll make the changes to docs-resources next.

wmat commented

The GitHub action build is now using all of these fixes in docs-resources. Should probably close this now and open new issues as you find them.

One thing I noticed is that the text inside tables is much smaller than the main body text. Was that intentional?

wmat commented
wmat commented

So table_font_size was changed to 11.5 recently and then overwritten back to 9 with the merging of the two themes. 9 seems a bit small to me. I'm thinking a size of ($base_font_size * 0.9em) would be appropriate. That is the equivalent to 10.35 pt when base font size is 11.5 pt.

That should work.

Thanks! That looked good!

One small nit if its not too hard to fix. I think the bullets are now a bit too big - I believe they are set to 1.5xbase-font - perhaps 1.25xbase-font is more appropriate. I pasted a google doc compared to the current render.
image

wmat commented

I actually am just using unicode glyphs. I believe that one is U-26AB. Does this look better? I can hardly see the difference.

Screenshot from 2024-06-12 14-22-32

wmat commented

i can't get it to change :/

Maybe it will respond to a "font-size" tag in ulist?

ulist:
marker:
disc:
content: "\u26ab"
circle:
content: "\u26ab"
square:
content: "\u25a0"
font-family: JetBrainsMono
font-size: ????

wmat commented

that's exactly what I tried.

Maybe U+2022 or U25CF is better?

I tried a bunch of these and seems like what you have currently is the best among all.