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

Sharing buttons on both Country and Measurement pages #476

Merged
merged 2 commits into from
Sep 30, 2020

Conversation

kronaemmanuel
Copy link
Contributor

Things to be discussed:

  • Design changes?
  • We can include a default text with the link on twitter. Should we include something like: "Check out this measurement on OONI Explorer: {link goes here}" ?

@hellais hellais self-assigned this Aug 25, 2020
@kronaemmanuel
Copy link
Contributor Author

Country Page sharing buttons:
image

Measurement Page sharing buttons:
image

@hellais
Copy link
Member

hellais commented Aug 26, 2020

Great job on this!

I have some suggestions about the design.

The issues I see with the current approach are:

  1. The icons are a bit hidden and they don't stand out much
  2. It's not clear what is going to happen when you click on them, because of 1., but also because the context might also reference social media itself (ex. we are talking about the blocking of the website twitter.com or the blocking of the facebook app)
  3. I am not a fan of us promoting the logos of facebook and twitter
  4. We should be absolutely sure users understand we will not track them and this is not the facebook/twitter evil tracking button, but our own tracker free option!

To this end I propose the following designs which I think address all the points above:

StreamingIconCountry2

StreamingIconMeasurement

The share icon should be inside of the material design icon set and it's called share-variant.

Let me know if you have any questions about the implementation.

@hellais
Copy link
Member

hellais commented Aug 26, 2020

TODO, following call with @sarathms:

  • Ensure that no external assets are loaded from facebook on first load.
  • Write copy for the boilerplate text in the share on facebook or twitter action (cc @agrabeli )
  • How does this behave on mobile? Is the design responsive?
  • What happens on mobile when you click on the links, does it trigger the app or what type of workflow occurs?

@hellais
Copy link
Member

hellais commented Aug 26, 2020

Wrong issue. See: #480 (comment)

@kronaemmanuel
Copy link
Contributor Author

Completed the changes. Here are the new screenshots:
Country Desktop:
image

Country Mobile:
image
image

Measurement Desktop:
image

Measurement Mobile (this whole page could use some redesign on mobile btw):
image

Copy link
Member

@hellais hellais left a comment

Choose a reason for hiding this comment

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

This is great work @kronaemmanuel ! Thanks for putting it together.

I checked the screenshots and it looks good to me!

@sarathms can you deploy this to our testing instance of OONI Explorer so I can do some functional testing as well, before we deploy it in production?

@sarathms
Copy link
Contributor

sarathms commented Sep 2, 2020

Deployed. Note that it had to be merged with the branch using test API server.

@sarathms
Copy link
Contributor

sarathms commented Sep 2, 2020

Also the social media previews may not look as good without the other metatag changes. Especially on Facebook.

@sarathms
Copy link
Contributor

sarathms commented Sep 8, 2020

@hellais Did you see the preview on explorer-test? Shall we merge this and consider tweaks for another PR?

Copy link
Contributor

@sarathms sarathms left a comment

Choose a reason for hiding this comment

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

Ready to merge 🐳

@sarathms sarathms merged commit 6d3e781 into ooni:master Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants