[keyserver] Use correct font for Comm title
ClosedPublic
Actions

Authored by tomek on Apr 6 2023, 3:21 PM.

Details

Reviewers

bartek
kamil

Commits

rCOMM8f3b91a09c19: [keyserver] Use correct font for Comm title

Summary

Get fonts url, link it to the page and use in a header.

Depends on D7312

Test Plan

Check if the font is used. Inspect web debugger and check if the fonts file is fetched.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Apr 6 2023, 3:21 PM

Herald added subscribers: atul, ashoat. · View Herald TranscriptApr 6 2023, 3:21 PM

tomek edited the summary of this revision. (Show Details)Apr 6 2023, 3:22 PM

Harbormaster completed remote builds in B18052: Diff 24738.Apr 6 2023, 3:38 PM

tomek requested review of this revision.Apr 6 2023, 3:38 PM

ashoat added inline comments.Apr 6 2023, 6:07 PM

keyserver/src/responders/website-responders.js
465 ↗	(On Diff #24738)	Personally I would add a `return` here, so you can reduce the indentation on the code below

tomek added a child revision: D7348: [keyserver] Add background and logo images.Apr 7 2023, 2:22 PM

kamil accepted this revision.Apr 11 2023, 9:04 AM

This revision is now accepted and ready to land.Apr 11 2023, 9:04 AM

Rebase

keyserver/src/responders/website-responders.js
465 ↗	(On Diff #24738)	Addressed in the diff D7312 that introduced this `if`

Harbormaster completed remote builds in B18217: Diff 24953.Apr 11 2023, 11:31 AM

Closed by commit rCOMM8f3b91a09c19: [keyserver] Use correct font for Comm title. · Explain WhyApr 12 2023, 1:48 PM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM8f3b91a09c19: [keyserver] Use correct font for Comm title.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

website-responders.js

5 lines

Diff 25040

View Options

keyserver/src/responders/website-responders.js

[keyserver] Use correct font for Comm titleClosedPublicActions