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

kickstart dynamic og images #1344 #1608

Closed
wants to merge 1,196 commits into from

Conversation

vasfvitor
Copy link
Contributor

Kickstart dynamic og images generation based on reference provided in #1344 with minor fixes

This depends on startlight being updated to 0.11.0. So it sorta depends on #1601

This is not ready as some pages don't have description field in the frontmatter, but have title. In those cases it is returning undefined written on the OG image. Example: /features/commands/

Plus the current design is minimal

preview:
commands
notification

renovate bot and others added 30 commits December 23, 2022 11:06
…uri-apps#1046)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…tauri-apps#1048)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [json5](https://github.com/json5/json5) from 2.2.1 to 2.2.2.
- [Release notes](https://github.com/json5/json5/releases)
- [Changelog](https://github.com/json5/json5/blob/main/CHANGELOG.md)
- [Commits](json5/json5@v2.2.1...v2.2.2)

---
updated-dependencies:
- dependency-name: json5
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…tauri-apps#1065)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…1063)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…#1068)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…1060)

* Add extra prerquisite in order to develop under windows

* Implement review feedback

* Update docs/guides/getting-started/prerequisites.md

I agree with your proposed change.

Co-authored-by: Fabian-Lars <[email protected]>

* Update docs/guides/getting-started/prerequisites.md

Co-authored-by: Lorenzo Lewis <[email protected]>

* at link to trunk

* Update docs/guides/getting-started/prerequisites.md

Co-authored-by: Lorenzo Lewis <[email protected]>

Co-authored-by: Fabian-Lars <[email protected]>
Co-authored-by: Lorenzo Lewis <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [ua-parser-js](https://github.com/faisalman/ua-parser-js) from 0.7.31 to 0.7.33.
- [Release notes](https://github.com/faisalman/ua-parser-js/releases)
- [Changelog](https://github.com/faisalman/ua-parser-js/blob/master/changelog.md)
- [Commits](faisalman/ua-parser-js@0.7.31...0.7.33)

---
updated-dependencies:
- dependency-name: ua-parser-js
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tauri-apps#1071)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* meilisearch evaluation

* correctly group categories

* dry-run automatic index update

* use api key from secret

* temporarily use default searchbar css

* throw together some fixes for our custom css

* don't overflow/no-wrap title

* Delete algolia.html

* revert SearchBar/index.js changes

* typo

* Switch to meilisearch cloud!!!

* test scraper upload

* revert test

* update netlify action

* line clamp

* don't include function signature in text selector
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Add version 2 alpha 3 release post

* Fix typo

* Mention exact version

---------

Co-authored-by: Wu Yu Wei <[email protected]>
* Rename 2023-02-03-tauri-2-0-0-alpha-3 .mdx to 2023-02-03-tauri-2-0-0-alpha-3.mdx

* Update author image

---------

Co-authored-by: Wu Yu Wei <[email protected]>
simonhyll and others added 19 commits September 15, 2023 23:30
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* update linux arm instructions, closes tauri-apps#1539

* add instructions to fix/prevent issues with main packages
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* feat: smooth bg gradient fade

port of tauri-apps#1381 to stable docs

* add z-index to heroContainer
…#1584)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [postcss](https://github.com/postcss/postcss) from 8.4.14 to 8.4.31.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.14...8.4.31)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#1596)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for tauri-docs-starlight failed.

Name Link
🔨 Latest commit 13420cf
🔍 Latest deploy log https://app.netlify.com/sites/tauri-docs-starlight/deploys/6526b43257f5ab0007050811

@github-actions github-actions bot added the i18n Is this related to translations? label Oct 11, 2023
@vasfvitor
Copy link
Contributor Author

build fail could be related to 80452ee return modification

@lorenzolewis
Copy link
Member

Hey @vasfvitor this is so awesome, thank you! I got #1601 merged in so this should be unblocked now. I took a few minutes to mock up a potential design for this based on our current direction if you'd like to take a stab at implementing it:

https://www.figma.com/file/ZgVWAeFQsLbFhUpuQ2kl17/Tauri.app-2.0?type=design&node-id=987%3A5394&mode=design&t=DbZoItgt9mEy7Myg-1

Open Graph Image_ Short

https://www.figma.com/file/ZgVWAeFQsLbFhUpuQ2kl17/Tauri.app-2.0?type=design&node-id=992%3A2233&mode=design&t=DbZoItgt9mEy7Myg-1

Open Graph Image_ Long

https://www.figma.com/file/ZgVWAeFQsLbFhUpuQ2kl17/Tauri.app-2.0?type=design&node-id=992%3A2255&mode=design&t=DbZoItgt9mEy7Myg-1

Open Graph Image_ Blog

I think we'll have two different types: 1 for pages, and another for a blog post. I've mocked up a design for each. Let's follow the reference for frontmatter for pages where title is required and description is optional (so we should be able to handle no description https://starlight.astro.build/reference/frontmatter/). Similar for blog posts but where title is required and excerpt is optional (https://github.com/HiDeoo/starlight-blog#frontmatter). I think it would also be good to include the date for a blog post


Open to feedback on any of this and thanks for getting this started! 💜

@vasfvitor vasfvitor closed this Oct 11, 2023
@github-actions github-actions bot removed the i18n Is this related to translations? label Oct 11, 2023
@vasfvitor
Copy link
Contributor Author

git mishap, workin on it

@vasfvitor
Copy link
Contributor Author

vasfvitor commented Oct 11, 2023

@lorenzolewis I made a mistake (synced with dev branch) but I solved now. I couldn't figure how to fix this pr showing the history from the 'dev' branch tho. I didn't intend to close the pr

Anyways... I'll start to work right away. Thanks for all the information and details

@lorenzolewis
Copy link
Member

No worries, we've all been there 😅 Feel free to open another PR when you're ready and just mention this one in it so we have a reference back.

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.