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

Landing page update #142

Merged
merged 24 commits into from
Oct 24, 2023
Merged

Landing page update #142

merged 24 commits into from
Oct 24, 2023

Conversation

LanesGood
Copy link
Member

Opening this as a draft until lad content is finalized.

Copy link
Member

@j08lue j08lue left a comment

Choose a reason for hiding this comment

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

The text could use a more thorough rework, replacing some of the very implementation-specific descriptions (e.g. FastAPI) with more function-oriented lingo (e.g. REST API service).

Not what I have done in my suggestions here - I just swapped out a few words I stumbled over.

docs/src/overrides/home.html Outdated Show resolved Hide resolved
docs/src/overrides/home.html Outdated Show resolved Hide resolved
docs/src/overrides/home.html Outdated Show resolved Hide resolved
docs/src/overrides/home.html Outdated Show resolved Hide resolved
@zacharyDez
Copy link
Contributor

The text could use a more thorough rework, replacing some of the very implementation-specific descriptions (e.g. FastAPI) with more function-oriented lingo (e.g. REST API service).

@j08lue Lane focused on structure; we can change the content as needed. I'll look into how we can include a functional description. Should it be in the bold gray subtitle like below:

image

The intent of the first bold title was to be functionaly driven at a higher level - Catalog your data, make it discoverable, visualize it. The newer grey subtitles might provide what we deem a functional description for more technical audiences? It seems safe to assume a developer interest in EO/geo understands what tiling means.

@zacharyDez
Copy link
Contributor

Regarding logos for the trusted by section, @j08lue @aboydnw @kcarini, can you let me know here or via Slack the steps we need to get approval or if we already have it?

@aboydnw
Copy link
Member

aboydnw commented Oct 10, 2023

@zacharyDez I can ask on getting approval for the NASA logo, we might just have to submit a form linking to how we want to use it

@zacharyDez
Copy link
Contributor

@j08lue, do the changes match your expectations on re-wording the content? Any other feedback on it?

Add top, left, and transform properties to center tout images in center of orange circles.
@zacharyDez
Copy link
Contributor

@kcarini @olafveerman @ianschuler @vincentsarago @sharkinsspatial @j08lue; could you give a second look when you can? I want to keep the ball rolling here and merge when we get approval for the logos.

@zacharyDez zacharyDez marked this pull request as ready for review October 16, 2023 21:15
@zacharyDez
Copy link
Contributor

zacharyDez commented Oct 16, 2023

Thanks to @vincentsarago 🙏 , the new version is deployed here: https://eoapi-dev1.surge.sh/

@vincentsarago vincentsarago merged commit 163f9c4 into main Oct 24, 2023
1 check passed
@vincentsarago vincentsarago deleted the landingpage-update1 branch October 24, 2023 15:25
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.

6 participants