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

UX improvements to snippet preview #13192

Closed
hedgefield opened this issue Jul 1, 2019 · 12 comments
Closed

UX improvements to snippet preview #13192

hedgefield opened this issue Jul 1, 2019 · 12 comments

Comments

@hedgefield
Copy link
Contributor

hedgefield commented Jul 1, 2019

Is your feature request related to a problem? Please describe.

The current toggle to switch between the mobile and desktop snippet preview is unclear. Based on the discussion about UX and a11y in this design issue, we have an approved product solution to build.

Describe the solution you'd like

snippet

cc @IreneStr

@IreneStr
Copy link
Contributor

IreneStr commented Jul 1, 2019

The radio buttons are based on those from Material UI. You can take some inspiration from their repo: https://github.com/mui-org/material-ui/tree/3f56ea14dbb964578eebe4258b50b2a0426c8cfb

@IreneStr
Copy link
Contributor

IreneStr commented Jul 1, 2019

@hedgefield During the scrum meeting, we decided to not collapse the edit fields by default, because that pushes the SEO analysis down to much.

@IreneStr IreneStr added the 3 label Jul 1, 2019
@Yoast Yoast deleted a comment from uxkai Aug 23, 2019
@Yoast Yoast deleted a comment from hedgefield Aug 23, 2019
@luckickken
Copy link

luckickken commented Aug 23, 2019

@IreneStr, @thijsdevalk and I decided a while ago to replace the radio buttons for a toggle switch. But after some consideration, I decided that this use case isn't suited for a toggle switch after all. For now, we'll stick to using radio buttons.

@IreneStr IreneStr removed the blocked label Oct 14, 2019
@Djennez Djennez added the Yoast: Content Preview Yoast Feature label Oct 14, 2019
@hwinne hwinne self-assigned this Oct 14, 2019
@hwinne
Copy link
Contributor

hwinne commented Oct 15, 2019

The radio buttons will not be styled like the material-ui components within this issue. After some consideration, for consistency's sake, the decision was made that a component should be created for all radio buttons, not only the ones in the scope of this issue. A separate issue will be created to redesign all radio buttons. In this issue default radio buttons will be used.

(CC @luckickken )

@hwinne
Copy link
Contributor

hwinne commented Oct 22, 2019

This issue has a PR in both the javascript and wordpress-seo repo.

@afercia afercia self-assigned this Oct 28, 2019
@afercia
Copy link
Contributor

afercia commented Oct 28, 2019

Rename it to Google search result preview

Testing #13649 I'm wondering how this new title would look like in other languages, typically the ones that have longer translations.

For example, in Italian Google search result preview

would be translated to:

Anteprima dei risultati della ricerca su Google

Especially in the sidebar, the available space is pretty limited:

Screenshot 2019-10-28 at 10 30 04

With this specific sentence, I think most of the western languages will have longer translations.

I'd suggest to consider a shorter title, which I guess should be evaluated by product.
/Cc @thijsdevalk @IreneStr

@afercia
Copy link
Contributor

afercia commented Oct 28, 2019

Oh well it's singular so it would be:
Anteprima del risultato della ricerca su Google

@afercia
Copy link
Contributor

afercia commented Oct 28, 2019

I see in the design issue a first idea was to rename the title to "Google preview", which is way shorter and maybe more immediate.

https://github.com/Yoast/design/issues/394#issuecomment-497673161

We should rename the Snippet preview to "Google preview", we're mimicking Google, so let's call it what it is.

@afercia
Copy link
Contributor

afercia commented Oct 28, 2019

One more question for product: if we rename "Snippet preview", I guess it should be renamed in the settings as well (2 occurrences in the codebase):

Screenshot 2019-10-28 at 11 16 12

@thijsdevalk
Copy link
Contributor

Lets go with Google preview.

@afercia Could you make a separate issue to replace the occurrences of Snippet preview in our codebase? Thanks!

@afercia
Copy link
Contributor

afercia commented Oct 28, 2019

Sure will do.

@afercia afercia removed their assignment Oct 28, 2019
@IreneStr IreneStr added 0.5 and removed 3 labels Dec 2, 2019
@igorschoester igorschoester self-assigned this Dec 9, 2019
@igorschoester igorschoester assigned hwinne and unassigned igorschoester Dec 9, 2019
@hwinne hwinne removed their assignment Dec 16, 2019
@abotteram abotteram assigned abotteram and unassigned abotteram Dec 17, 2019
@IreneStr IreneStr self-assigned this Dec 17, 2019
@IreneStr
Copy link
Contributor

Fixed in Yoast/javascript#376 and #13649

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants