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

feat(web): Update information copy and CTAs #7568

Merged
merged 11 commits into from
Dec 20, 2024

Conversation

tonypls
Copy link
Collaborator

@tonypls tonypls commented Dec 17, 2024

Issue

Users perceive us as an NGO

Description

This PR updates the app description and about sections on the ranking panel and info modal to better explain the value provided by electricity maps. Removes clutter from the ranking panel and makes "Access the API" CTA's more discoverable

"text": "The Electricity Maps app provides real-time insights on web and mobile for citizens, energy experts, NGOs and policymakers promoting a deeper understanding of energy transition challenges.",
"open-source-text": "This app is <a href='{{link}}' target='_blank'>Open Source</a> and powered by the community.",
"title": "About Electricity Maps",
"text": "Electricity Maps is an API platform providing real-time and predictive electricity signals allowing any device to reduce their cost and emissions by informing them about the best time to consume electricity.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Small tweak suggestions to the grammar for clarity:

Electricity Maps is an API platform that provides real-time and predictive electricity signals, enabling devices to reduce costs and emissions by identifying the best times to consume electricity.

Copy link
Contributor

@cadeban cadeban Dec 19, 2024

Choose a reason for hiding this comment

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

Also, I wonder if we should link to the commercial website here (at least for the time being). "To find out more..." etc etc

Copy link
Contributor

Choose a reason for hiding this comment

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

Or just link the words "Electricity Maps" to the commercial website?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll try the link! I'm a little bit cautious to change the copy for this one in particular as it's copy pasta from else where (job posting I believe) and having them drift slightly might be confusing

Copy link
Contributor

Choose a reason for hiding this comment

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

Fair enough!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think it looks great!
image

Thanks for the suggestion!

web/src/locales/en.json Outdated Show resolved Hide resolved
Copy link
Member

@madsnedergaard madsnedergaard left a comment

Choose a reason for hiding this comment

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

Good comments by @cadeban, except for utm params I have nothing more to add :)

<MethodologyCard />
<HorizontalDivider />
<div className="mb-1 text-sm font-semibold">Looking for forecast data?</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

"Looking for forecast data" and the CTA feel a little disconnected. Maybe they could go on the same line? 🤔
Screenshot 2024-12-19 at 10 53 22

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I agree, this was quite a speculative change, we can remove it or try on the same line

@cadeban
Copy link
Contributor

cadeban commented Dec 19, 2024

Left a couple of small comments, but otherwise it's looking good!

@tonypls tonypls merged commit 5837128 into master Dec 20, 2024
23 checks passed
@tonypls tonypls deleted the tonyvanswet/tig-26-update-in-app-descriptions branch December 20, 2024 03:55
<div className="pb-5">
<h1>{t('ranking-panel.title')}</h1>
<h2 className="text-sm">{t('ranking-panel.subtitle')}</h2>
<p className="text-sm">{<Trans i18nKey="ranking-panel.subtitle" />}</p>
Copy link
Member

@VIKTORVAV99 VIKTORVAV99 Dec 31, 2024

Choose a reason for hiding this comment

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

FYI this little change increased the bundle size by 7.3 kb and that the subtitle is constantly re-rendering on a datetime change.

I am fixing it in: #7647

Oh and this also broke the Title size flow on the ranking panel as there is no h2 anymore but there is a h1 and h3. This is bad for SEO and accessibility.

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