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

add a thanks page to our doc #28744

Draft
wants to merge 25 commits into
base: main
Choose a base branch
from
Draft

add a thanks page to our doc #28744

wants to merge 25 commits into from

Conversation

Johann-S
Copy link
Member

@Johann-S Johann-S commented May 8, 2019

  • Improve the style
  • Add Netlify
  • List OC sponsors/backers by Tier
  • Switch to a plain list with just the images?
  • Fix removed/obsolete core classes usage
  • Add the amount of each person's contribution as the image title?
  • Add direct buttons to OC for each Tier
  • Reword page header
  • Fix page description
  • See if we need markdownify or chomp in any of those fields
  • Move inline styles to css
  • Move inline JS to a separate JS file or HTML include, preferably the first
  • Add page link in the sidebar
  • Have this in the homepage instead of a separate page?
  • Check everything in dark mode

Live previews

@Johann-S Johann-S added the docs label May 8, 2019
@XhmikosR XhmikosR force-pushed the jo-docs-thanks-page branch 2 times, most recently from 5144138 to 2161ae9 Compare May 8, 2019 13:52
@MartijnCuppens
Copy link
Member

Changed the theming a bit and did some copy changes.

@Johann-S
Copy link
Member Author

Johann-S commented Jun 3, 2019

Hi @znarf do you know when this endpoint: https://opencollective.com/bootstrap/members/all.json?TierId=7192 will have CORS too ?

@Johann-S Johann-S force-pushed the jo-docs-thanks-page branch 2 times, most recently from fabb694 to a3c807b Compare June 3, 2019 14:43
@znarf
Copy link

znarf commented Jun 3, 2019

@Johann-S https://opencollective.com/api/groups/bootstrap/backers is not documented and deprecated, this should not be used. We're trying to get rid of it. opencollective/opencollective#2037

I now recommend to start using the GraphQL API v2 for everything. https://medium.com/open-collective/open-collective-graphql-api-preview-3b42ed1d55ff

If what you need is a REST endpoint with CORS, let me know exactly the required data and I'll try to make that available.

@Johann-S
Copy link
Member Author

Johann-S commented Jun 5, 2019

Hi @znarf,

Thanks for letting me know that 👍 but we need a REST endpoint with CORS, we don't have a backend server which will fetch the OC data for us 😟 I would love using your GraphQL but it would expose our API key publicly

@znarf
Copy link

znarf commented Jun 5, 2019

Like I said:

If what you need is a REST endpoint with CORS, let me know exactly the required data and I'll try to make that available.

What the endpoint should be and which data should it returns? And on your side, which fields are you reading from?

@Johann-S
Copy link
Member Author

Johann-S commented Jun 5, 2019

The end point should be this one: https://opencollective.com/bootstrap/members/all.json with CORS and should accept a TierId query param to let us filter by tier id.

The data we need for our Backers and Sponsors:

  • Name
  • Image
  • Website
  • IsActive
  • TotalAmountDonated
  • TierId

@znarf
Copy link

znarf commented Jun 7, 2019

What do you think of that? https://rest.opencollective.com/v2/bootstrap/orders/incoming/active

It's featuring all active subscriptions, has CORS and all the fields you need. Only tierId is not available, better use tier.slug instead.

@Johann-S
Copy link
Member Author

Johann-S commented Jun 7, 2019

Thanks @znarf ! That's awesome 👍

I saw we can use this one too: https://rest.opencollective.com/v2/bootstrap/orders/incoming
To get all the people who supported us

@znarf
Copy link

znarf commented Jun 7, 2019

@Johann-S maybe you want then to filter the ones with status=ERROR (payment failed) and status=PENDING (payment waiting for confirmation).

@Johann-S
Copy link
Member Author

Johann-S commented Jun 7, 2019

I don't think it's usefull for us 🤔 what you provided to us is perfect 👍

@XhmikosR XhmikosR force-pushed the jo-docs-thanks-page branch 2 times, most recently from b3f87d9 to e434915 Compare June 8, 2019 11:55
@Johann-S
Copy link
Member Author

Johann-S commented Jun 11, 2019

Pushed a commit where we use the new API (thanks @znarf), we have a small CSS issue with backers images

@patrickhlauke
Copy link
Member

Is this still relevant?

@XhmikosR
Copy link
Member

XhmikosR commented May 4, 2021

Yeah, it's relevant although probably not like it is right now. We still need to add sponsors in the homepage later.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 9, 2022

Images automagically compressed by Calibre's image-actions

Compression reduced images by 16.4%, saving 84.00 KB.

Filename Before After Improvement Visual comparison
site/static/docs/5.2/assets/img/bootstrap-sponsoring.png 256.55 KB 214.55 KB -16.4% View diff
site/static/docs/5.2/assets/img/bootstrap-sponsoring@2x.png 256.55 KB 214.55 KB -16.4% View diff

105 images did not require optimisation.

Update required: Update image-actions configuration to the latest version before 1/1/21. See README for instructions.

@julien-deramond
Copy link
Member

julien-deramond commented Dec 9, 2022

Tried something in dbc7287 to also add a new section in our homepage. Done very quickly but you get the spirit 😇

/cc @mdo

Screenshot 2022-12-09 at 08 53 02

Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

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

Might need an update of the assets to migrate to v5.3 and update the markup to adopt some new helpers we have

Copy link
Member

Choose a reason for hiding this comment

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

This one look exactly the same as the one @2x, is that fine ?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, it should be changed whenever this PR is finalized. Let's keep this conversation opened to not forget it :)

site/layouts/partials/home/sponsoring.html Outdated Show resolved Hide resolved
@@ -0,0 +1,23 @@
<section class="row g-3 g-md-5 pb-md-5 mb-5 align-items-center">
<div class="col-lg-6">
<div class="masthead-followup-icon d-inline-block mb-3" style="--bg-rgb: var(--bd-violet-rgb);">
Copy link
Member

Choose a reason for hiding this comment

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

Don't we have any other color in here ? It looks like duplicate of the previous one

Copy link
Member

Choose a reason for hiding this comment

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

We already use danger, info, pink, primary, violet, teal, warning, violet on this page. The remaining colors in the palette are too close to the used one (green almost like teal, cyan like info, indigo like violet) so I'm not sure how to choose another one. @mdo could help us if we try to finalize this PR.

## Sponsors

{{< sponsors.inline >}}
<div id="sponsorList" class="d-flex flex-wrap mx-n2 text-center"></div>
Copy link
Member

Choose a reason for hiding this comment

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

Unless I'm wrong, Bootstrap doesn't provide negative margins in here so the rendering looks a bit shifted.
image

Copy link
Member

Choose a reason for hiding this comment

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

Yep, you're right, it was added in the first commits from this PR in 4c7953d. I'll check how to change the rendering.

Copy link
Member

Choose a reason for hiding this comment

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

Enhancement of the rendering done via d5b5c3b. Maybe there's a better solution, but this quick one should allow to resolve this thread.

site/content/docs/5.2/about/support-bootstrap.md Outdated Show resolved Hide resolved
@julien-deramond
Copy link
Member

If having a new homepage section to support Bootstrap is still a thing, I could extract this independent part from this PR so that it can be tackled probably more rapidly, while we take our time for the pure sponsors page to find the right technical solution.

Thoughts? (/cc @mdo)

Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

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

The rendering looks reaaly good, well done! Some minor changes before merging and we're good to go on my side.

{{ if .website -}}
<a href="{{ .website }}" class="stretched-link text-reset" title="{{ .name }}" target="_blank" rel="nofollow noopener noreferrer">
{{ end -}}
<img src="{{ printf ("/docs/%s/assets/img/services/%s") $.Site.Params.docs_version .image }}" alt="{{ .name }}" class="mh-100 mw-100">
Copy link
Member

Choose a reason for hiding this comment

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

Since they are static images, we have an issue with dark mode, maybe add a static background to its parent or use the svg directly (this way we can color the text using var(--bs-body-color) or something similar) ?

)
}

output.push(`<img src="${backerAccount.imageUrl}" alt="${backerAccount.name}" class="img-fluid d-block">`)
Copy link
Member

Choose a reason for hiding this comment

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

To have at least the backer's pseudo displayed somehow

Suggested change
output.push(`<img src="${backerAccount.imageUrl}" alt="${backerAccount.name}" class="img-fluid d-block">`)
output.push(`<img src="${backerAccount.imageUrl}" title="${backerAccount.name}" alt="${backerAccount.name}" class="img-fluid d-block">`)

output.push(
'<div class="position-relative">',
' <div class="img-thumbnail mx-auto d-flex align-items-center justify-content-center overflow-hidden sponsor">',
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" alt="${sponsorAccount.name}">`,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" alt="${sponsorAccount.name}">`,
` <img class="img-fluid d-block" src="${sponsorAccount.imageUrl}" title="${sponsorAccount.name}" alt="${sponsorAccount.name}">`,

)

if (sponsorAccount.website) {
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" title="${sponsorAccount.name}" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</a>`)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" title="${sponsorAccount.name}" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</a>`)
output.push(`<a href="${sponsorAccount.website}" class="stretched-link text-reset" target="_blank" rel="nofollow noopener noreferrer">${sponsorAccount.name}</a>`)

})()
})()
</script>
{{< /services.inline >}}
Copy link
Member

Choose a reason for hiding this comment

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

Feels wrong to have this closing tag after the script one

</p>
</div>
<div class="col-lg-6">
<img class="img-fluid mt-3 mx-auto" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png,
Copy link
Member

Choose a reason for hiding this comment

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

Better fit with general spirit of the page in dark mode imo

Suggested change
<img class="img-fluid mt-3 mx-auto" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png,
<img class="img-fluid mt-3 mx-auto rounded" srcset="/docs/{{ .Site.Params.docs_version }}/assets/img/bootstrap-sponsoring.png,

Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't we add a section with best or last donators in here ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs review
Development

Successfully merging this pull request may close these issues.

7 participants