This diff introduces the StatusIndicator component. We will use this component in several places, so I figured it would be good to factor this logic into its own component.
for context here are the designs:
Depends on D9887
Differential D9888
[web] introduce status indicator component ginsu on Nov 14 2023, 2:24 PM. Authored by Tags None Referenced Files
Details
This diff introduces the StatusIndicator component. We will use this component in several places, so I figured it would be good to factor this logic into its own component. for context here are the designs: Depends on D9887 Please see screenshot below active: inactive:
Diff Detail
Event TimelineComment Actions I kind of feel like the status indicator and the keyserver name look really far apart with a lot of blank space in between, but that's just my opinion regarding the design. Comment Actions
Going to land this diff for now since this concern is outside the scope of this diff/this interface is only visible to staff members/devs. However, I know we have implementing the web redesign in our roadmap, and I think that could help with the "blank space" feel. |