Sav22999 / websites-notes

Take notes on every website in a smart and simple way!
https://www.notefox.eu
Mozilla Public License 2.0
32 stars 8 forks source link

Can you add shortcuts and possibility to use '*' to add a note to all the matching urls? #13

Closed nik-gr closed 1 year ago

nik-gr commented 1 year ago

Extension shortcuts to open (a) domain notes, (b) page notes pop ups and (c) for opening the All notes page

About *: For example with github.com/Sav22999/ to display the specific note when any of the pages under this github account are visited Perhaps with an addition of an extra tab in the menu (next to "Domain" and "Page"). And by clicking there (or with shortcut) the popup will have the same place to write/read the related note but also one extra editable line maybe at the bottom with the url in an editable form so users can change it using '' and add the note to all the matching urls.

Sav22999 commented 1 year ago

I'll see what I can do 😄

About the shortcuts... you mean to open the popup to the specific tab or when you already opened the pop-up?

In addition, what do you mean "Open All notes page"? As before, a generic shortcut?

nik-gr commented 1 year ago

Thanks!

I mean to open the popup and to the specific tab (covers also the other case: if it's already open, with the other shortcut will open the other tab - or it won't?)

In addition, what do you mean "Open All notes page"? As before, a generic shortcut?

Yes, basically a shortcut for the place where we will be able to perform a search

Sav22999 commented 1 year ago

@nik-gr shortcuts added in 2.3 😄

nik-gr commented 1 year ago

Thank you!

There is one thing, as I understand when there is a note only for Domain or only for Page it will open that tab regardless the shortcut pressed. That can be a little confusing (but still easy to figure out by checking the title of the active tab) but more important it makes it impossible to write a 2nd note using the shortcuts (for example if there is a note for the domain and want to write an extra note for a specific url there, there is no way to open the page tab using the shortcuts)

And some thoughts you may want to consider There is no need for toggle the display, the popup note can be hidden just by pressing escape and actually it makes it a bit smoother experience without it (as it is now if I would like to move from Domain to Page tab or vice versa I need to type the shortcut twice - the 1st time it will hide it) Since there are shortcuts for each tab (thanks again) I don't see any usage of the shortcut for the default tab opening. For anyone using the shortcuts, the other 2 are sufficient (a welcome 3rd shortcut could be to open in a new browser tab the addon page with all the notes and even better with the focus on the search field)

Sav22999 commented 1 year ago

@nik-gr the "default" shortcut it's useful if you want to open the popup by the default choosing (and, in addition, you can close the popup with the same shortcut).

I'll add the feature to change tab by shortcut also when opened and I'll fix bugs

Sav22999 commented 1 year ago

Published the update (2.3.1) which fixes many bugs (the icons weren't updated "in real time", shortcuts, etc.)

nik-gr commented 1 year ago

still buggy: If there is a note for the domain it always opens the Domain tab regardless of shortcut (unlike before now it's happening even if there is a note also for the page)

Sav22999 commented 1 year ago

@nik-gr I cannot understand, sorry. Could you list me the steps to follow to get the error? Thanks

nik-gr commented 1 year ago

If I have a note written for the Domain then whenever I am in a webpage from that domain with any of the 3 shortcuts the addon popup always opens the "Domain" tab showing the note I wrote. If I also have a Page note for the specific webpage I cannot open the "Page" tab to read it using the shortcut (instead it will display the Domain tab or hide it). And of course I cannot use the Page note shortcut to write a Page note for any webpage under that domain

Sav22999 commented 1 year ago

I'll check why the add-on presents that issue. Have you update to the latest version the add-on, right?

nik-gr commented 1 year ago

Hi Saverio, with the new version it works as expected 👍

If it is an easy implementation it would be better if when the Domain/Page note is open and then use the shortcut for Page/Domain (=the shortcut of the other note type) to immediately change and open the other tab, as it is now it will hide the note and will display the other note after press the shortcut a 2nd time (but still it is functional so if it's not easy to implement the change it doesn't worth the effort)

Sav22999 commented 1 year ago

@nik-gr Now it should work in that way: if the addon (popup) is already opened you can switch tab via shortcuts. Doesn't it work to you? 🤔

nik-gr commented 1 year ago

Yes it does thank you!

my comment was that to be able to switch you need to press the shortcut twice (1st time it hides the already opened note) but it's not really of an issue

May I ask, are you considering to implement my other suggestion as well? (the one for notes using '*' in the url)

Sav22999 commented 1 year ago

@nik-gr about the shortcuts: you should already able to switch pressing just once the popup (if you get another behaviour maybe there is a bug... let me know, because to me it works great)

About the other feedbacks, I'm still thinking, but actually I don't think it could be so useful; I mean, I'm basing on the KISS principle, and I think this is a feature that could interest just a few users... maybe in future I'll implement although. I hope you can enjoy anyway this add-on, sorry

nik-gr commented 1 year ago

ok that's strange... for me if the popup is open the use of any shortcut will hide it