Quote option not available on multi click to select entire sentence #3603
Comments
|
I believe this is due to the selection technically being outside the post body, or some other strange thing. If I remember, triple clicking on a sentence that isn't the last one of a post works fine. |
|
You're right!
It works on all lines but the last one.
Very strange...
…On Mon, 15 Aug 2022, 15:19 David Wheatley, ***@***.***> wrote:
I believe this is due to the selection technically being outside the post
body, or some other strange thing.
If I remember, triple clicking on a sentence that isn't the last one of a
post works fine.
—
Reply to this email directly, view it on GitHub
<#3603 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABJXQ6JOAJ5GIWK3PW5H23VZI7VNANCNFSM56SEQNPQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
I can't reproduce this on the discuss Edit: See that now on Chrome |
|
alright it's a chrome issue |
|
To do my bit, I noticed something that might explain what is going on. Edge: Chrome: Firefox: Do you notice something strange? Well yes, in Firefox it works 😂, but the detail is that in Chromium based browsers (Edge/Chrome) they are adding a blank space (that doesn't exist) at the end of the sentence, maybe that's the reason why it fails? Or maybe this is just a visual rendering issue of Chromium and this I'm commenting is stupid, no idea... |
|
Interesting! You mean we need to report a bug against Chromium?
…On Thu, 25 Aug 2022, 02:02 Darkle, ***@***.***> wrote:
To do my bit, I noticed something that might explain what is going on.
Edge:
[image: Screenshot 2022-08-25 015226]
<https://user-images.githubusercontent.com/15818451/186544088-d7ffdf5b-79ff-4a57-aa47-a85611ff951d.png>
Chrome:
[image: Screenshot 2022-08-25 015300]
<https://user-images.githubusercontent.com/15818451/186544095-fdb81eab-51cc-41c4-8046-9f4427034788.png>
Firefox:
[image: Screenshot 2022-08-25 015339]
<https://user-images.githubusercontent.com/15818451/186544104-970a404a-a099-43d4-aaac-0842e5860039.png>
Do you notice something strange? Well yes, in Firefox it works 😂, but
the detail is that in Chromium based browsers (Edge/Chrome) they are adding
a blank space (that doesn't exist) at the end of the sentence, maybe that's
the reason why it fails?
Or maybe this is just a visual rendering issue of Chromium and this I'm
commenting is stupid, no idea...
—
Reply to this email directly, view it on GitHub
<#3603 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABJXQ75M5BCBMMLNITVXATV22ZXRANCNFSM56SEQNPQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
It turns out the selection start varies between browsers. When you select that last line on Chrome, the range gets undesired common ancestor (the entire post stream rather than the post body) where Firefox has the post body as its common ancestor. I'll take a look into it. |
|
Thanks so much, Hasan!
…On Thu, 25 Aug 2022 at 11:30, Hasan Özbey ***@***.***> wrote:
It turns out the selection start varies between browsers. When you select
that last line on Chrome, the range gets undesired common ancestor (the
entire post stream rather than the post body) where Firefox has the post
body as its common ancestor. I'll take a look into it.
—
Reply to this email directly, view it on GitHub
<#3603 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABJXQ5XFKCX4NL6FABCER3V244K7ANCNFSM56SEQNPQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Current Behavior
The quote option is not present when you do a multi click to select not just one word but the entire sentence.
Steps to Reproduce
Expected Behavior
I expect the quote option to be present
Screenshots
Video: https://drive.google.com/file/d/1wf__APRVnttZ8T1ukz4U-L-qPg-3XN4D/view?usp=drivesdk
Environment
Output of
php flarum infoNo response
Possible Solution
No response
Additional Context
No response
The text was updated successfully, but these errors were encountered: