Differential D12591

[landing] add link to download page in the header
ClosedPublic
Actions

Authored by ginsu on Thu, Jun 27, 10:20 AM.

Tags

None

Referenced Files

	F2197939: D12591.id.diff
	Fri, Jul 5, 4:16 PM

	F2197826: D12591.id41856.diff
	Fri, Jul 5, 3:59 PM

	Unknown Object (File)
	Thu, Jul 4, 10:22 AM

	Unknown Object (File)
	Thu, Jul 4, 4:21 AM

	Unknown Object (File)
	Wed, Jul 3, 10:08 PM

	Unknown Object (File)
	Wed, Jul 3, 2:17 PM

	Unknown Object (File)
	Wed, Jul 3, 2:17 PM

	Unknown Object (File)
	Wed, Jul 3, 12:38 PM

View All 14 Files

Subscribers

Details

Reviewers

inka
ashoat

Commits

rCOMM1ca7214148fa: [landing] add link to download page in the header

Summary

We need to introduce a new button in the header that links to the download page I just introduced

Depends on D12583

Test Plan

Please see the demo video below

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ginsu created this revision.Thu, Jun 27, 10:20 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Jun 27, 10:20 AM

ginsu edited the summary of this revision. (Show Details)Thu, Jun 27, 10:23 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: inka, ashoat.

ginsu added a parent revision: D12583: [landing] introduce download page.

ginsu attached a referenced file: F2128288: Screen Recording 2024-06-27 at 1.21.08 PM.mov. (Show Details)

ginsu added inline comments.Thu, Jun 27, 10:25 AM

landing/header.css
75 ↗	(On Diff #41760)	Decided to make the hover effect for the web app icon and the download icon be the same purple circle so rename this to be a bit more generic

ginsu added a child revision: D12592: [landing] add download icon to mobile nav.Thu, Jun 27, 10:28 AM

Harbormaster completed remote builds in B29962: Diff 41760.Thu, Jun 27, 10:36 AM

ginsu requested review of this revision.Thu, Jun 27, 10:36 AM

Can you add a video of what the dropdown menu looks like when the width is decreased? I want to make sure the download link appears there too

EDIT nevermind that's in D12592

This revision is now accepted and ready to land.Mon, Jul 1, 12:12 PM

rebase before landing

This revision was landed with ongoing or failed builds.Mon, Jul 1, 12:31 PM

Closed by commit rCOMM1ca7214148fa: [landing] add link to download page in the header (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM1ca7214148fa: [landing] add link to download page in the header.

Harbormaster completed remote builds in B30032: Diff 41856.Mon, Jul 1, 12:47 PM

Revision Contents
Changeset List

Path

Size

landing/

10 lines

header.react.js

19 lines

landing.react.js

15 lines

Diff 41859

landing/header.css

Loading...

landing/header.react.js

Loading...

landing/landing.react.js

Loading...