Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(marketing): add nudge to Postman #995

Merged
merged 1 commit into from
Dec 2, 2020
Merged

Conversation

LoneRifle
Copy link
Contributor

Problem

Fixes #985

Solution

  • add a line of text to prompt the user to consider Postman
  • line up the state switch to flush with the top of text indicator

Screenshots

image

Copy link
Contributor

@liangyuanruo liangyuanruo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as commented. could you also confirm that the mobile view is OK?

@NatMaeTan
Copy link
Collaborator

Is there a reason why the "mass messaging tool" part was taken out?

@LoneRifle LoneRifle force-pushed the chore/marketing/postman branch from 5c12d0e to 485b726 Compare December 2, 2020 02:19
@LoneRifle
Copy link
Contributor Author

as commented. could you also confirm that the mobile view is OK?

the screenshot is the mobile view.

@LoneRifle
Copy link
Contributor Author

Is there a reason why the "mass messaging tool" part was taken out?

The sentence isn't grammatically correct. I can try to incorporate it though.

- add a line of text to prompt the user to consider Postman
- line up the state switch to flush with the top of text indicator
@LoneRifle LoneRifle force-pushed the chore/marketing/postman branch from 485b726 to c1a0db2 Compare December 2, 2020 03:46
@LoneRifle LoneRifle merged commit df489e5 into develop Dec 2, 2020
@LoneRifle LoneRifle deleted the chore/marketing/postman branch December 2, 2020 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Insert a note on link drawer about sharing through Postman
4 participants