Richard Ortiz

Results 14 comments of Richard Ortiz

I still need to have a look at some tests, but I'm moving this to needs review so that we can move forward.

> The CSS file is being fetched, but for some reason the styles it has aren't being applied. I've tried a bunch of stuff but haven't got anywhere. I'm going...

Closing this PR in favor of: - https://github.com/Automattic/wp-calypso/pull/92188

If I navigate away from `/wp-admin?admin-interface-changed=true` the tour isn't displayed/completed. Maybe we can add an option or cookie to check if the tour is in progress and continue to show...

When the notice is displayed, and I move to step 2 and then reload, the tour starts at step 1 again.

> > If I navigate away from `/wp-admin?admin-interface-changed=true` the tour isn't displayed/completed. Maybe we can add an option or cookie to check if the tour is in progress and continue...

[Minor] When we resize the screen the current tour step doesn't adjust to the new screen size until re-rendered. https://github.com/Automattic/jetpack/assets/1989914/a0abc141-494f-47d6-893b-f0d05b18199e

> (3) the tour will still be there for those who navigate away and go back to the dashboard. This statement is misleading, by `go back` you mean navigating using...

> The code "works" now. **It means that `wp_navigation` posts will not be exported if they were authored by a user no longer on the site.** I am somewhat concerned...

> @javierarce I have some mixed feelings about using a modal instead of an on-page nudge: > > * Codewise we'll need a conditional to either display the upgrade nudge...