The current reference superscript is rendered like normal text.
Also superscripts aren't actionable in the share a fact cards.
Have we already tried to eliminate them?
Description
Details
Related Objects
- Mentioned In
- rAPIW9fdcbc39e13c: refactor & add more sanitation to article snippet heuristic
rAPIWc73627349e01: fix section share snippet XPath to only grab top-level <p>'s
rAPIWbd8d68ebb30b: do text sanitation on selected text before sharing - Mentioned Here
- T98626: 4.1.3(109) Cannot use a custom text selection in 'Share fact'
Event Timeline
@Dbrant, I just verified this on Android and you are absolutely right. Removing Android App from this.
Thanks!
Change 209163 had a related patch set uploaded (by Bgerstle):
do text sanitation on selected text before sharing
Change 209298 had a related patch set uploaded (by Bgerstle):
fix section share snippet XPath to only grab top-level <p>'s
Change 209298 merged by Mhurd:
fix section share snippet XPath to only grab top-level <p>'s
@BGerstle - can you attach one more screenshot where the sentence has a reference superscript?
The first sentence in Madame Clinton's article has no superscripts.
Sorry to be a prude :(
@Vibhabamba sorry I posted that screenshot on the wrong ticket. You should be able to verify this and the other share a fact bug in today's alpha build (should be 107). Ping me when you get in if you need more info.
@BGerstle Ideally it doesn't matter what I select - I should not see numbers in the card.
It mostly works unless I really mess with it, which is possible for users.
Text selection is always jumpy, never accurate.
See if you can handle case 2 (Screenshot from the time article).
Just mess around with what you select in lead section.
@Bgerstle based on the other 3 screenshots, 99% of the times, i don't see superscripts.
This is signed off by design.
Re-checked the samples given by @Vibha - no superscripts are displayed in 4.3.1(109). However, I filed T98626
4.1.3(109) Cannot use a custom text selection in 'Share fact' - could be related to the fix?