-
Notifications
You must be signed in to change notification settings - Fork 336
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
Investigate re-adding instructions "attachment" to exit this page button #3267
Comments
From a design point of view it seems simple and clear enough. My only query would be can we size up to 16px for the font size or does it have to be 14px? I understand about the screen size meaning that we might have scale down but I think even on a small screen we might be able to get away with 16px due to it going across the wrong screen. If this isn't possible, just let me know. |
@Ciandelle Sorry, just to confirm, the guidance text is already 16px on desktop resolutions and the text is currently hidden on narrow screens where it would normally be scaled down to 14px. Assuming you intended the '16' size on our font-scale (which is 19px on desktop), here's a mockup of that: Personally, I think this makes the supporting text look a little overwhelming and too prominent compared to the button; but maybe there's a way to resolve that, like changing the text to a shade of grey? |
I saw about the narrower screens after I wrote this as I looked at them in the wrong order, my bad. I think that it would be great to have them the same size but in grey. However as I've mentioned on the #3268 I think the longer text would work better. So if we need to keep the original size I completely understand. |
Needs a @davidc-gds review. |
Here are my vague thoughts so far, in no particular order:
That last thought is my least coherent, but it might be something worth looking at: what's the value of having the instructions on the 'skip link' styled EtP button? Is it more valuable there compared to on the visual button? Less valuable? Equally valuable? Seems there's a bit more to mull over. |
We can definitely do this as part of the community crit, if we feel this will be useful. In the past I've done some really basic A/B testing in crits to see if the thing I'm doing is useful to the vast majority of users.
|
@davidc-gds Thanks for the input. Firstly, a thought on the original intent behind this. This was to solve the problem of how we provide accurate instructions to people based on the small chance they might not have js enabled. If we're saying "press shift 3 times" but the js to manage that function isn't available, the user, hypothetically, could be at risk. I'm therefore wary of the prospect of showing the instructions by default. This rings true for the skip link as well as the interruption page instructions. With the skip link there's also a risk that we're "hiding" this content until someone navigates to it. The other side of that is that, firstly, we can serve a class to only show content if js is enabled and ask teams to add this in the pattern guidance. Although this might have content consistency issues. The content being on the skip link also makes sense as you say because the audience is keyboard users. It's not critical to the function of the button. It's difficult for me to assess if this is something that would be useful to non-keyboard only users or what the overall impact of just keeping this content by default would be. That leads me to another broader point: it does seem like this needs more verification. The community crit will be useful but also potentially a round of user research. I don't think this should mean we delay launch as we can investigate and iterate the component after an initial publish. A/B testing I'm not sure about as we currently don't have a way to measure specific component features in services from where we are. There may be an opportunity here to work with a team who would be using the component and getting them to A/B test the feature? |
@owenatgov this definitely feels like a case for get out the MVP and then iterate it. I think as long as we make sure the guidance is clear and states that we are doing further testing on it, then we shouldn't be delaying the launch. From a personal point of view I think that we try without. I think that because we mention it in the interruption page, and the fact that there may be more than one interruption page, we may be adding too much to the button. However we can have this as one of the questions in the crit if that would help? |
Okay, my thoughts. Trying to write this through everyone's different mental models. Suggestions in summary:
Full message: Agree with that the button is the primary way to use Exit this page. And to prioritise keeping that clear and simple visually. Telling the user how to use the secondary way ("Shift key x3") would be nice to have underneath the button, since it'd be visible on every page and give some nice context being nearby, but it's feeling like too complicated a message within the confines of what should be a small call-to-action. From what I'm hearing, the frontend, design and content constraints are tight and possibly butting up against one another. Agree that the interruption page feels like the best place to put all instructions for all 3 ways to Exit this page. Appreciate the Javascript-disabled risk, but we'd ideally want to tell the user about this anyway, in case they use the same service on different devices, etc. Open-minded about also putting "Shift key x3" instructions alongside the hidden skip link, but feeling it's too long a message to be effective. And also slightly confusing (exit this page now vs later)? Is this effectively a hidden notification banner? |
It seems that the consensus is to not merge this enhancement at this time. We still have open questions, but it sounds like we'd rather investigate those once we're sure that the core parts of EtP are working well, and may revisit this enhancement in future. If there's no objections, I'll close this issue and the associated PR soon. |
What
Investigate adding a visual attachment to the exit this page button which lets the user know they can activate the button by pressing shift 3 times. This attachment should only be visible and accessible when js is active for the user.
Why
This is a potential solution to the problem of how we communicate to users that features of the EtP button such as activating by pressing shift 3 times aren't available when js is broken or turned off. The hypothesis is that by attaching it to the component, we make it easier to manage these instructions as opposed to having to ask service teams to include a class around content in their service as part of the EtP pattern.
We included this feature in a previous prototype and would need to reassess why we remove this, if there was a specific reason. We'd additionally need to assess if this design was suitable against the EtP button in it's current state.
Part of alphagov/govuk-design-system#1755
Timebox
1 working week (5 days) from start
Who is working on this?
@owenatgov @querkmachine @calvin-lau-sig7 @Ciandelle
Spike lead:
@owenatgov
Questions to answer
Done when
The text was updated successfully, but these errors were encountered: