Adding an icon so the web app (https://web.comm.app) can be launched from the landing page.
Note: Won't land this diff until https://web.comm.app is live
Details
Details
Here's what the header looks like after:
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| landing/header.react.js | ||
|---|---|---|
| 49–51 ↗ | (On Diff #10255) | Not really sure if it will be intuitive for users that this icon opens our web app |
| landing/header.react.js | ||
|---|---|---|
| 49–51 ↗ | (On Diff #10255) | Agreed. I think something like "open app" or even just app would be easier. Or possibly just signin/signup?    |
Comment Actions
I spoke w/ @ashoat, and I think for now we kind of want to keep it subtle/don't want to advertise it too much
Comment Actions
The icons feel a bit too far apart, can they be moved closer?
And yeah I want to keep this understated... we're not trying to advertise the fact that this is live.
Comment Actions
The icons feel a bit too far apart, can they be moved closer?
Yeah, they're space-between here so it looks too far apart on desktop and too close together on mobile. Should be addressed in https://phabricator.ashoat.com/D3404 where we switch to flex-end and have constant space between the icons.