Page MenuHomePhabricator

[web] Restyle `LoginForm` inputs to match designs
ClosedPublic

Authored by atul on Feb 2 2023, 2:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 7:16 PM
Unknown Object (File)
Sun, Apr 21, 7:16 PM
Unknown Object (File)
Sun, Apr 21, 7:16 PM
Unknown Object (File)
Sun, Apr 21, 7:16 PM
Unknown Object (File)
Sun, Apr 21, 7:16 PM
Unknown Object (File)
Sun, Apr 21, 7:12 PM
Unknown Object (File)
Mar 31 2024, 12:32 PM
Unknown Object (File)
Mar 19 2024, 7:56 AM
Subscribers

Details

Summary

Restyle the text/password inputs in LoginForm to match the designs.

Test Plan

Before:

After:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.Feb 2 2023, 2:21 PM
atul edited the test plan for this revision. (Show Details)
ashoat added inline comments.
web/theme.css
69 ↗(On Diff #21917)

Would be better if we introduced a name for this color or something but whatever

This revision is now accepted and ready to land.Feb 3 2023, 8:54 AM
web/theme.css
69 ↗(On Diff #21917)

As in create a new color called like btn-bg-filled-new or something and use it everywhere btn-bg-filled was?

The change in theme.css also changed the color of some of the other buttons. For example:

image.png (154×1 px, 17 KB)

the "Create new chat" now has a different color and doesn't match the settings cog.

web/theme.css
69 ↗(On Diff #21917)

Nah I mean like --violet-dark-100