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

v97 #1346

Merged
merged 14 commits into from Feb 13, 2022
Merged

v97 #1346

merged 14 commits into from Feb 13, 2022

Conversation

Thorin-Oakenpants
Copy link
Contributor

@szepeviktor
Copy link

szepeviktor commented Feb 5, 2022

0605: see bugzilla.mozilla.org/show_bug.cgi?id=1355460

Mozilla guys are very tricky!

@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented Feb 5, 2022

Mozilla guys are very tricky!

I don't see an issue. About the only time I think this would trigger without you wanting to actually visit is when you left click to drag and drop. I only added it to be consistent

edit: to be clear this is for existing bookmarks, not creating them

@Thorin-Oakenpants
Copy link
Contributor Author

am done with #1366 and ready to merge - awaiting reviews

user.js Outdated Show resolved Hide resolved
user.js Outdated Show resolved Hide resolved
@GlassGruber
Copy link

Mozilla guys are very tricky!

I don't see an issue. About the only time I think this would trigger without you wanting to actually visit is when you left click to drag and drop. I only added it to be consistent

Wouldn't this be triggered when reordering bookmarks?

@Thorin-Oakenpants
Copy link
Contributor Author

Wouldn't this be triggered when reordering bookmarks?

IDK and don't care, I disabled it :) I use a speed dial for my stuff, otherwise my toolbar has

  • three folders and then four bookmarks - i never need to reorder anything - the rest of the space on the toolbar if for drg n drop URLs for me to follow up on etc
  • i never use the bookmarks manager

So that's what I meant when I said drag n drop. I have no idea what you mean by "reordering bookmark" and how that differs from what I said

@GlassGruber
Copy link

GlassGruber commented Feb 11, 2022

So that's what I meant when I said drag n drop. I have no idea what you mean by "reordering bookmark" and how that differs from what I said

  • i never use the bookmarks manager

I do sometimes use the bookmark manager and the usability... well is not really the main sell point of the tool IMO. It happens sometimes that I need to rearrange bookmarks that have been saved in weird order or similar.

IDK and don't care, I disabled it :)

Disable first, ask questions later. Sounds like a good plan. 😎

@GlassGruber
Copy link

Wouldn't this be triggered when reordering bookmarks?
I have no idea what you mean by "reordering bookmark" and how that differs from what I said

Mmm I messed it a bit, I actually really intended a question but that negation morphed it in an assertion, correct phrase is:

Would this be triggered when reordering bookmarks?

Anyway yea, disabled and done.

user.js Show resolved Hide resolved
@Thorin-Oakenpants Thorin-Oakenpants merged commit a98b73c into master Feb 13, 2022
@crssi
Copy link

crssi commented Feb 13, 2022

❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants