Thanks to visit codestin.com
Credit goes to github.com

Skip to content

New first run wizard#960

Merged
szaimen merged 1 commit intomasterfrom
feature/new-first-run-wizard
Nov 2, 2023
Merged

New first run wizard#960
szaimen merged 1 commit intomasterfrom
feature/new-first-run-wizard

Conversation

@marcoambrosini
Copy link
Member

@marcoambrosini marcoambrosini commented Oct 6, 2023 •
edited by susnux
Loading

Close #888

TODO:

  • re-add video and automatically move to second slide after it is finished?
  • shorten the video/audio?
  • check with marketing if all messages are in
  • change Other Downloads to Calendar and contacts with subline Connect your calendar and contacts with your devices
  • Audio doesn't work
  • Fix sync clients settings section
  • max-contrast-text add version number (maybe as subline in last slide=
  • on page 2-3 circle on top should only be line like in text
  • allow to navigate slides with arrow buttons?
  • ask for final a11y review
  • Re-add 3 seconds webm video
  • address comments
  • fix styling of video (should have rounded corners)
  • Fix styling of buttons in settings (currently broken on master as well)
  • fix layout problems (shrinking, mobile)
  • squash
  • rebase
  • test on different screen resolutions
  • fix tests
  • merge
Sneakpeak
Aufzeichnung.2023-11-02.131725.mp4

@szaimen szaimen added this to the Nextcloud 28 milestone Oct 6, 2023
szaimen
szaimen reviewed Oct 6, 2023
View reviewed changes
szaimen
szaimen reviewed Oct 6, 2023
View reviewed changes
@szaimen szaimen mentioned this pull request Oct 6, 2023
Closed
@marcoambrosini marcoambrosini force-pushed the feature/new-first-run-wizard branch from 7c3d73b to 07928d2 Compare October 11, 2023 07:58
szaimen
szaimen reviewed Oct 11, 2023
View reviewed changes
szaimen
szaimen reviewed Oct 12, 2023
View reviewed changes
szaimen
szaimen reviewed Oct 12, 2023
View reviewed changes
szaimen
szaimen reviewed Oct 12, 2023
View reviewed changes
Copy link
Collaborator

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

some comments

szaimen
szaimen reviewed Oct 12, 2023
View reviewed changes
@jancborchardt
Copy link
Member

jancborchardt commented Oct 17, 2023

@schiessle so at some point we added the version number to the first slide so people easily find it. With the new design we want to keep it a bit cleaner, would it be ok if we have the version number only on the last/third slide?

@jancborchardt
Copy link
Member

jancborchardt commented Oct 18, 2023

@szaimen @marcoambrosini regarding the video point:

re-add video and automatically move to second slide after it is finished?

Thought about it a bit more and yes, let’s add it back and automatically move on to the second slide when it is finished. @szaimen regarding your point that it will not be themed:

  • The default experience is still the main goal, and that looks much better with the video
  • It is some advertisement for Nextcloud which is nice
  • If people want to theme it out or rather replace it with their own video they can
  • If we really want to make it look nice and themed, we could do a CSS animation in the beginning. This is something for the future though, nothing immediate. :) (Also it has to be done very well for it to look as good as the video.)

@schiessle
Copy link
Member

schiessle commented Oct 18, 2023

@schiessle so at some point we added the version number to the first slide so people easily find it. With the new design we want to keep it a bit cleaner, would it be ok if we have the version number only on the last/third slide?

would be fine for me.

@marcoambrosini marcoambrosini force-pushed the feature/new-first-run-wizard branch from 9dff0a3 to 9b03a8f Compare November 1, 2023 08:19
@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
Merged
@marcoambrosini
Copy link
Member Author

marcoambrosini commented Nov 1, 2023

@nextcloud/a11y-review could you please give us an accessibility review of this? Thanks :)

ShGKme
ShGKme reviewed Nov 1, 2023
View reviewed changes
@marcoambrosini marcoambrosini force-pushed the feature/new-first-run-wizard branch from 258b4aa to c0079b9 Compare November 2, 2023 09:08
@marcoambrosini marcoambrosini marked this pull request as ready for review November 2, 2023 09:10
szaimen
szaimen reviewed Nov 2, 2023
View reviewed changes
tests/Controller/WizardControllerTest.php
Comment on lines -108 to -112
['customclient_desktop', 'https://nextcloud.com/install/#install-clients', 'https://nextcloud.com/install/#install-clients'],
['customclient_android', 'https://play.google.com/store/apps/details?id=com.nextcloud.client', 'https://nextcloud.com/install/#install-clients'],
['customclient_fdroid', 'https://f-droid.org/packages/com.nextcloud.client/', 'https://nextcloud.com/install/#install-clients'],
['customclient_ios', 'https://geo.itunes.apple.com/us/app/nextcloud/id1125420102?mt=8', 'https://nextcloud.com/install/#install-clients'],
['appstoreenabled', true, true]
Copy link
Collaborator

@szaimen szaimen Nov 2, 2023

Choose a reason for hiding this comment

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

@marcoambrosini I fixed the tests. However we should probably re-introduce this so that customers can adjust the links. Should we do this still in this PR or rather in a follow-up like the transitions?

Copy link
Collaborator

@szaimen szaimen Nov 2, 2023

Choose a reason for hiding this comment

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

Put it into #985

@szaimen szaimen force-pushed the feature/new-first-run-wizard branch from 15f3410 to 5d3d8f1 Compare November 2, 2023 10:14
szaimen
szaimen reviewed Nov 2, 2023
View reviewed changes
@szaimen szaimen force-pushed the feature/new-first-run-wizard branch from 5d3d8f1 to 53ac2d7 Compare November 2, 2023 12:06
@szaimen szaimen mentioned this pull request Nov 2, 2023
Closed
2 tasks
@szaimen szaimen force-pushed the feature/new-first-run-wizard branch from 53ac2d7 to 65fd174 Compare November 2, 2023 12:42
@szaimen
Copy link
Collaborator

szaimen commented Nov 2, 2023

/compile amend /

@marcoambrosini @nextcloud-command
New first-run wizard
40c0881 
Signed-off-by: Marco <[email protected]>
Signed-off-by: Simon L <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command force-pushed the feature/new-first-run-wizard branch from 65fd174 to 40c0881 Compare November 2, 2023 14:03
@szaimen szaimen disabled auto-merge November 2, 2023 14:52
@szaimen
Copy link
Collaborator

szaimen commented Nov 2, 2023

CI failure unrelated

@szaimen szaimen merged commit 84b3a24 into master Nov 2, 2023
@szaimen szaimen deleted the feature/new-first-run-wizard branch November 2, 2023 14:53
@szaimen
Copy link
Collaborator

szaimen commented Nov 2, 2023

🎉🎉🎉🎉🎉

nickvergessen
nickvergessen reviewed Nov 10, 2023
View reviewed changes
img/user.svg
Copy link
Member

@nickvergessen nickvergessen Nov 10, 2023

Choose a reason for hiding this comment

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

File is still used in notifications which are currently broken…

image not found: image:user.svg webroot: serverroot:/var/www/…

Was deletion intentional?
Will restore in a new PR.

@ChristophWurst ChristophWurst mentioned this pull request Jul 31, 2024
Closed
@szaimen szaimen mentioned this pull request May 12, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessen nickvergessen nickvergessen left review comments

@ShGKme ShGKme ShGKme left review comments

@szaimen szaimen szaimen approved these changes

Assignees

@marcoambrosini marcoambrosini

@szaimen szaimen

Labels

3. to review design enhancement

Projects

None yet

Milestone

Nextcloud 28

Development

Successfully merging this pull request may close these issues.

Update design (and possibly content) of first run wizard slides

6 participants

@marcoambrosini @jancborchardt @schiessle @szaimen @nickvergessen @ShGKme