Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Gif replace #519

Merged
merged 6 commits into from
Feb 27, 2025
Merged

Gif replace #519

merged 6 commits into from
Feb 27, 2025

Conversation

novalisdenahi
Copy link
Contributor

@novalisdenahi novalisdenahi commented Feb 11, 2025

Description

The Jira, monday.com, VSCode, Trello and IntelliJ v2 docs gifs are replaced with mp4. In the case of IntelliJ, the V1 doc is the same, so I replaced the gifs there as well.

Trello card

link to the Trello card

Notes for QA

The Jira, monday.com, VSCode, Trello and IntelliJ v2 docs gifs are replaced with mp4. In the case of IntelliJ, the V1 doc is the same, so I replaced the gifs there as well.

Requirement checklist

  • I have validated my changes on a test/local environment.
  • I have tested that the code snippets I added work. (Leave unchecked if there are no new code snippets.)
  • I have added my changes to the V1 and V2 documentations.
  • I have checked the SNYK/Dependabot reports and applied the suggested changes.
  • (Optional) I have updated outdated packages.

Copy link

cloudflare-workers-and-pages bot commented Feb 11, 2025

Deploying configcat-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 38248ff
Status:⚡️  Build in progress...

View logs

@novalisdenahi novalisdenahi added the ux-review The PR needs a review from the UX team label Feb 21, 2025
@novalisdenahi novalisdenahi marked this pull request as ready for review February 21, 2025 15:19
@novalisdenahi novalisdenahi requested a review from a team as a code owner February 21, 2025 15:19
Copy link

Head's up @configcat/ux-qa-team - someone needs a UX review!

Copy link

Deploying configcat-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 38248ff
Status: ✅  Deploy successful!
Preview URL: https://bc4a56a4.docs-xmy.pages.dev
Branch Preview URL: https://repalce-gifs.docs-xmy.pages.dev

View logs

@adams85
Copy link
Contributor

adams85 commented Feb 25, 2025

From a UX point of view, it all looks good. 👍

One thing I noticed is that in the VS Code guide, the top of some of the images is cropped, which doesn't look very nice:

image

However, this issue existed in the original gifs, so it's ok with me if we deal with this later, not in the current PR.

@adams85 adams85 added the ux-approved When a UX member approves a PR label Feb 25, 2025
@novalisdenahi
Copy link
Contributor Author

I created a ticket for the VSCode docs update so we won't forget.

@novalisdenahi novalisdenahi merged commit 764dd02 into master Feb 27, 2025
7 checks passed
@novalisdenahi novalisdenahi deleted the repalce-gifs branch February 27, 2025 13:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ux-approved When a UX member approves a PR ux-review The PR needs a review from the UX team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants