oughtinc / ice

Interactive Composition Explorer: a debugger for compositional language model programs
https://ice.ought.org
MIT License
529 stars 66 forks source link

Fix content centering inside CallIconButton #309

Closed rickycheah closed 1 year ago

rickycheah commented 1 year ago

For the Button component, className changed form w-12 to w-fit to allow enough margin numbers with higher digit count.

image

For elements inside the Button, added h-4 to normalize the height of the button because ChatCenteredDots and "𝑓" have different heights.

When showing ChatCenteredDots or "𝑓", added mx-1 to add margin and center the element horizontally.

image

The "𝑓" symbol itself is not centered vertically (see image in link), added position: relative; to center it vertically.

<img width="350" alt="image" src="https://github.com/oughtinc/ice/assets/49166991/d5da90b8-a644-4b7a-be62-a479324d2cb9">

github-actions[bot] commented 1 year ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

rickycheah commented 1 year ago

I have read the CLA Document and I hereby sign the CLA

stuhlmueller commented 1 year ago

Thank you!