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

Issue with Link Tooltop in Firefox #600

Closed
nickian opened this issue Jan 21, 2020 · 11 comments
Closed

Issue with Link Tooltop in Firefox #600

nickian opened this issue Jan 21, 2020 · 11 comments
Labels
1. to develop bug Something isn't working design Experience, interaction, interface, … good first issue Good for newcomers

Comments

@nickian
Copy link

nickian commented Jan 21, 2020

NextCloud version: 17.0.2
Text version: 1.1.1
Firefox: 72.0.1

I can't add links to text in Firefox. This is what it looks like. When I try to click the tooltip it just goes away.

Screen Shot 2020-01-21 at 1 11 10 PM

As a side note, it would also be nice to have a source editing option in the browser, not just wysiwyg.

@nickian nickian added the bug Something isn't working label Jan 21, 2020
@juliusknorr juliusknorr added 1. to develop design Experience, interaction, interface, … good first issue Good for newcomers labels Jan 24, 2020
@ahstro
Copy link
Member

ahstro commented Feb 25, 2020

Is this still happening and/or can someone else reproduce it? I tried in both Firefox 73.0.1 and Firefox Developer Edition 74.0b4, but it worked fine, and being on Arch, I can't easily install a previous version.

@nickian
Copy link
Author

nickian commented Feb 25, 2020

Yes, still an issue for me on Windows 10 and MacOS, Firefox 73.0.1. The input comes up, but doesn't look right. It's cut off on the left side and just doesn't behave the same as Chrome.

34234234

@ahstro
Copy link
Member

ahstro commented Feb 25, 2020

Hmm, that looks like a different issue to me, specifically the menu bubble not showing up correctly on too narrow editor windows. I'll look into that right away 😊

But in which way is it behaving differently? I'll try to help with that as much as possible, but unfortunately(?) I'm getting the same behavior from all my browsers 😅

@ahstro
Copy link
Member

ahstro commented Feb 25, 2020

The alignment issue seems to be an upstream one: ueberdosis/tiptap#239
It is allegedly fixed, but since that doesn't seem to be the case here, I asked the creator of the original PR.

Still, if something other than the alignment is off, share! 😊

@nickian
Copy link
Author

nickian commented Feb 25, 2020

On Mac OS, I am not able to get the textbox to appear:

text

On Windows, it works, but if I'm trying to add a link that happens to be on the left side of the document, the popup ends up going too far left and getting cut off. Maybe some conditional logic that says if the selected text is x pixels away from the left edge, then increase left positioning by however much is needed to not cut off.

@nickian
Copy link
Author

nickian commented Feb 25, 2020

Actually the left positioning issue seems to be universal. This is in Chrome on Mac OS, as an example:

text2

@nickian
Copy link
Author

nickian commented Feb 25, 2020

Also, I can't help but wonder, why not just have the link icon in the menu with the rest of the items instead of this tooltip approach? Seems inconsistent and arbitrary.

@ahstro
Copy link
Member

ahstro commented Feb 26, 2020

This app is using tiptap for the editor, so I'm assuming the Nextcloud-people didn't make a lot of the design decisions, but yeah, I agree that it seems arbitrary. It's also in tiptap where the left alignment issue is.

I don't have access to a Mac, so I can't look into that issue, unfortunately :(

@juliusknorr
Copy link
Member

This app is using tiptap for the editor, so I'm assuming the Nextcloud-people didn't make a lot of the design decisions, but yeah, I agree that it seems arbitrary. It's also in tiptap where the left alignment issue is.

tiptap is a renderless library, which means how the UI looks and acts is actually the responsibility of the text app. The modal for the link was actually decided that way together with @jancborchardt .

@jancborchardt
Copy link
Member

Yep, the floating bar could have other inline items as well like bolding and italic next to the link.

Anyhow – that’s a separate discussion. The issue should of course be fixed, and the bar should show properly inside the available viewport. :)

@azul
Copy link
Contributor

azul commented Sep 29, 2020

I think this is fixed in NC 20:

  • The second menu entry "Link to a file" - made the initial popup wider
  • Fix link menu positioning #1045 fixed the remaining.
    Closing this for now. Feel free to reopen if it's still showing up in NC 20+

@azul azul closed this as completed Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug Something isn't working design Experience, interaction, interface, … good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

5 participants