==== Problem
Because the link inspector is not dismissable by the user, it's possible that it can cover up article text that they would want to see when deciding on their suggestion. This edge case occurs on desktop in certain scenarios such as when only one suggestion is in the article (because the inspector can't go away, and the user can't even flip to a different suggestion to expose the obscured text for the other suggestion) and when the link suggestion is towards the very bottom of article content (where users can't scroll underneath the inspector).
Even when users can read article text by switching to another, there is certain level of lost context needed to switch to the another link inspector in order to read the text around the link suggestion being reviewed.
Example of the link inspector covering part of the sentence that contains the suggestion:
{F34428830}
Note: Since the issue on mobile is not about text being obscured but the annoyance of limited scrolling area, a separate task for mobile at T284609
==== User job story
**When I** am reviewing suggested link text and part of the text below the suggestion is covered by the link inspector card,
**I want to** be able to easily see the obscured text below the link inspector card,
**So that** I can get better context about what the link suggested relates to, and decide whether to accept or reject that link suggestion.
==== Proposed solutions
=====Proposal A. Enable moving the link inspector
Add an action that enables the user to drag the link inspector to a different location. If the user moves onto another link, the inspector returns to its normal position next to the link text.
Notes on design:
- When the link inspector is moved, the callout arrow pointing to the text is hidden, to prevent the inspector from inadvertently pointing to unrelated text
- For easier mobility, enable the entire header section of the inspector to be dragged, not only the move icon
| 1. Inspector with move icon {F34725769} | 2. Inspector is moved to another part of the screen {F34725773}
- **Pros**
-- User has control of where the inspector should go
-- Recovery by going to the next or previous suggestion
-- Maintains context with the link inspector connected to the link text
- **Cons**
-- Additional control to the UI adds complexity and effort (esp. in comparison to proposal B)
-- Potential for inspector to be moved unwittingly to the wrong link suggestion
-- Potential for unexpected bugs and edge cases (moving inspector to an unrecoverable area, inspector not returning to the normal position, etc)
-- Potential a11y issues with drag
=====Proposal B. An explicit re-open button when the link inspector is hidden
Enable the user to hide the link inspector completely when tapping on a close button or inside the article content area.
Add a button that allows the inspector to be re-opened.
This is the same as proposal B on Mobile (T284609)
| 1. User taps on the close {F34725777} | 2. Link inspector is transitions to hidden the screen {F34725779} | 3. Link inspector can be brought up again by tapping on the Floating button with the robot on the bottom right (or if the user taps on a link suggestion tag in the article). {F34725782}
A simplified version of this could be to not include the button, but toggle show/hide the inspector any time the user clicks on the article content area.
- **Pros**
-- Users are familiar with clicking outside of pop-ups to dismiss them
-- Re-open button acts as an additional safeguard in case newcomers do not know how to re-open the inspector via clicking on the suggested link text again
-- Maintains context with the link inspector connected to the link text
-- Can work in concert with proposal to allow moving the inspector (the close icon would not be shown in this case)
- **Cons**
-- Potential for conflicts with auto-advancing behaviour (e.g., user closes as the inspector as it auto advances)
!! Proposal A & B could potentially both be offered to enable different users the flexibility to either hide or move the inspector. !!
=====[[ https://www.figma.com/file/rmLdUNo2A9wti2lMeB2BLI/T281185-and-Link-inspector-obscures-text?node-id=88%3A11 | Desktop mocks on figma ]]