Page MenuHomePhabricator

[landing] introduce keyserver faq component
ClosedPublic

Authored by ginsu on Jun 5 2023, 6:51 AM.
Tags
None
Referenced Files
F3409185: D8093.id27440.diff
Wed, Dec 4, 3:11 PM
F3408830: D8093.id29761.diff
Wed, Dec 4, 2:00 PM
F3407398: D8093.diff
Wed, Dec 4, 6:01 AM
F3406299: D8093.diff
Wed, Dec 4, 1:19 AM
Unknown Object (File)
Sun, Nov 17, 2:18 AM
Unknown Object (File)
Sun, Nov 17, 1:39 AM
Unknown Object (File)
Sun, Nov 17, 1:39 AM
Unknown Object (File)
Sun, Nov 17, 1:39 AM
Subscribers

Details

Summary

This diff is the first part to introduce the keyserver faq component. The focus of this diff is to get the general structure/behavior of the keyserver faq. Subsequent diffs will handle other things like the animation and implementation of this component

Depends on D8092

Test Plan

Please see the demo video/screenshots below

Desktop/Responsive:

Tablet:

Screenshot 2023-06-05 at 10.00.18 AM.png (2×3 px, 903 KB)

Mobile:

Screenshot 2023-06-05 at 10.00.01 AM.png (2×3 px, 930 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 5 2023, 6:59 AM
Harbormaster failed remote builds in B20020: Diff 27440!
ginsu requested review of this revision.Jun 5 2023, 8:06 AM

Looks good, think we can skip classNames altogether for headingClassName though.

landing/keyserver-faq.react.js
13

Do we need to be using classNames at all here?

This revision is now accepted and ready to land.Jun 6 2023, 8:08 AM
This revision was landed with ongoing or failed builds.Aug 8 2023, 2:09 PM
This revision was automatically updated to reflect the committed changes.