search

ScottLogic / prompt-injection

Application which investigates defensive measures against prompt injection attacks on an LLM, with a focus on the exposure of external tools.
MIT License
13 stars 9 forks source link

fixing radio button centering #857

Closed AAloshine-scottlogic closed 4 months ago

AAloshine-scottlogic commented 4 months ago

Description

added the same styling of the toggle switches to the radio buttons for focusing and hovering, recentered the selected radio button dot

Screenshots

before

image

after

image image

note: maybe text align to the left so the overflow text on the next line isn't as close to the highlight? or i could increase the margin from 0.5 to 0.6rems to give the radio button a little more space?

Checklist

Have you done the following?

chriswilty commented 4 months ago

@AAloshine-scottlogic I'd say we put the radiobuttons and labels into some sort of container - flex or grid - so that when the text wraps, it looks like this:

radiobuttons

Also. just check that the radio dot is still centered on various zoom levels, as that was an issue previously. Thanks!

chriswilty commented 4 months ago

@pmarsh-scottlogic Just waiting on a re-review from you, then this one can go in.