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: connect studio tour to editor tour #346

Merged
merged 4 commits into from
Oct 4, 2024

Conversation

deltork
Copy link
Collaborator

@deltork deltork commented Oct 1, 2024

PR Goal?

Connecting studio tour to the editor tour

Fixes?

n/a

Feedback sought?

sanity check

Priority?

low

Tests added?

n/a

How to test?

Do the full tour in the pr preview

Confidence?

High

Version change?

patch

Copy link

semanticdiff-com bot commented Oct 1, 2024

Review changes with SemanticDiff.

Analyzed 7 of 7 files.

Overall, the semantic diff is 13% smaller than the GitHub diff.

Filename Status
✔️ packages/studio-web/src/i18n/messages.es.json 40.0% smaller
✔️ packages/studio-web/src/i18n/messages.fr.json 40.0% smaller
✔️ packages/studio-web/src/i18n/messages.json 40.0% smaller
✔️ packages/studio-web/src/app/app.component.html Analyzed
✔️ packages/studio-web/src/app/shepherd.steps.ts 2.35% smaller
✔️ packages/studio-web/src/app/studio/studio.component.ts Analyzed
✔️ packages/studio-web/src/app/editor/editor.component.ts Analyzed

Copy link
Contributor

github-actions bot commented Oct 1, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-04 14:45 UTC

@joanise
Copy link
Member

joanise commented Oct 2, 2024

This is OK, but it would be nice to bring the user's attention up to the Editor button in the top-right corner. Can we have an extra step in the Studio tour on the Editor button, with something like:
Title: Go to the Editor
Text: Once you've downloaded your ReadAlong, you can reload it into the Editor to continue editing it.
Here you could have the Finish and Editor buttons to finish or continue on to the Editor tour.

@deltork deltork force-pushed the dev.del/join-studio-and-editor-tours branch from 6e63b4a to 3905e4a Compare October 3, 2024 14:15
@deltork
Copy link
Collaborator Author

deltork commented Oct 3, 2024

@joanise, please add translations for fr and es.

@deltork
Copy link
Collaborator Author

deltork commented Oct 4, 2024

Thanks

Copy link
Member

@joanise joanise left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@joanise joanise merged commit 8b89a03 into main Oct 4, 2024
2 checks passed
@joanise joanise deleted the dev.del/join-studio-and-editor-tours branch October 4, 2024 14:44
joanise added a commit that referenced this pull request Oct 16, 2024
* feat: connect studio tour to editor tour

* chore: updated translations

* fix: implemented feedback from Eric

* chore: translations for the tour connection to the editor

---------

Co-authored-by: Eric Joanis <eric.joanis@nrc-cnrc.gc.ca>
joanise added a commit that referenced this pull request Oct 16, 2024
* feat: connect studio tour to editor tour

* chore: updated translations

* fix: implemented feedback from Eric

* chore: translations for the tour connection to the editor

---------

Co-authored-by: Eric Joanis <eric.joanis@nrc-cnrc.gc.ca>
joanise pushed a commit that referenced this pull request Oct 18, 2024
* feat: connect studio tour to editor tour

* chore: updated translations

* fix: implemented feedback from Eric

* chore: translations for the tour connection to the editor

---------

Co-authored-by: Eric Joanis <eric.joanis@nrc-cnrc.gc.ca>
(cherry picked from commit 8b89a03)
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.

2 participants