A little bit late but thanks a lot for the review. Really glad that you like it.

Update: Sorry but I was not able to finish the update yet. I will update the message once it's done.
-------------------------
Hi, and thanks a lot for the detailed information.
Sadly everything you mentioned is correct and the reason is that I implemented the saving mechanism without taking into consideration that closed windows should be saved. I realized that the behavior is not what is normally expected from a auto-saving feature. It should store all windows no matter how many are opened and closed and that's something that I quite failed to implement properly.
I will try to fix this ASAP (I should be able to fix it this weekend) and latest by Monday a new version to be ready.
Regarding saved location. Sadly the auto-saving functionality is not stored into Bookmarks. They are saved into the browser "localStorage" that is hidden from users. Most addons out there will store everything into this storage and you won't have access to the data unless the addon offers some kind of an export functionality. I decided to store what user would consider important data into "Bookmarks" because they users will be in control of it and not the addon and data will be there (visible) always. Please understand that not saving everything into bookmarks is not an issue but the other way around. I tried to offer something better by saving only those things that matter to you (every user) in the most transparent way possible (through bookmarks).

The reason why auto-saved data can't be stored in bookmarks:
- because it is saved periodically and saving into Bookmarks would create performance issues for many users as well as possible sync errors (bookmarks are not really intended to be changed that often, I mean not once every 10 seconds)
- bookmarks are usually intended for things that you want to keep for a longer period of time
- auto-saved sessions are usually intended only as recovery options/backup restoring: browser crashing, system crashing or getting back to a certain site that you know you visited a few hours ago/days

Because of this decision only bookmarks are synced across Firefox accounts. But I would say that is ok because you can always decide if you want to promote an auto-saved session to a synced session by saving it into Bookmarks.

Usually there are many things to consider when building such an extension and I tried to keep a good balance between functionality and features. It won't be perfect for everyone but as I receive feedback I try to improve it. Thanks again for feedback. Promise that I will have an update for this ASAP. I will update this comment as soon as the implementation is ready.

Hi, and thanks for feedback. I don't understand exactly what exactly it's happening in your case because saving multiple windows definitely works. Not sure if there can be any other addons that can cause such an issue but saving definitely works ok. Of course... there can be lots of things that I can't control that can affect the addon but the code does exactly what it's suppose to. I will investigate to see if I can replicate this but I can definitely say that I never seen encounter this kind of bug.
The addon may not be for everyone and I do get that but please understand that your experience with it most probably is not reflection of the addon quality. 99.99% of the time the addon works as it should I would say. If there are serious issues that I can tackle (like the one you've mention) I will try to fix them as soon as possible but in this case I simply have no lead what exactly causes Firefox to stop saving the second window.
If you are able to provide more information that would be helpful (I usually don't test everything on linux and I don't have a Mac so if it happens to be one of these 2 I might have a better lead) please send me an email. Of course, I will still try to figure out what may be the cause for this issue.

Favicon request code was removed (as of version 3.0.3). No single request is created from within the addon toward a third-party server. Favicons are directly taken from the sites that you visit (so each favicon from it's respective site and cached locally if possible).

----------------------------------------------------
Hi. I care really a lot about privacy and I really messed up this. Since there was no way around to get links favicons using WebExtensions I looked up for alternatives and I found that google has a service for this. I'm really really sorry about this. When I read your message I literately stopped for 2 seconds realizing how idiot I've been that I thought this is an alternative solution. Really really sorry for doing this. And no, of course I'm not paid by google. I simply made a stupid decision without realizing that service may be tracked by google.
I really understand your concern and it's totally right.
Again.. really really sorry. The code will be removed right away. I will update the message as soon as I fix this.

Reply to P.S.S.S:
=)) Nobody's annoying. I should be really grateful for so much helpful feedback. Although we should have probably not used this review section as a chat. Feel free to contact me on my mail. You can find a leave feedback button with my mail on the top-right side of the help page.

Sadly that "Current Session" from the auto-save section is actually a snapshot of the current session that can be from anywhere between 1 second ago and 10 minutes (whatever time you set in the configuration). So it's not really a reflection of your current session. Again... probably creates some confusions this one as well but I assumed since it's in the auto-saved session it shouldn't be that problematic. Most sure I was wrong. Yes it's sort of funny but like I said when you actually try to fit everything right, there's always something that is not really ok.

-----------------------------------------------
Reply to P.S.S
Yes sadly it does that... it does changes the right panel (where it shows the active session instead of a certain selected session if you had one selected) but I do understand. I don't really like it how it is right know. I'll definitely try to find a solution to it.

------------------------------------------------
Reply to P.S
You can that that too with SessionSync (CTRL + SHIFT+ V) will open it into a Window or a Tab (depending on what you have set from configurations).
Session Buddy is a great session manager... true, but at the same time quite different from this. I actually use (and probably other users as well) Session Sync as a great organized bookmark manager: If I find something interesting I add the current tab to a certain session. You can only do this by keeping the current tab active only by using the the panel.
The truth is... although I have some feedback I truly have no idea how many users use it purely as a Session Manager, how many as a Bookmarking Manager and how many as both. All I know is that I have somehow to find good solutions (while also preserving a good user experience) to fit as many individual expectations as possible :D. And that's incredibly hard to do. But feedback is great and I will try to improve it as much as possible.

------------------------
Later edit:
1. It's true that Active Session is duplicate with the one from 3rd tab - "Current Session" but the 2nd tab does not do that. Basically the first tab is there just to press it to get fast to the "Current Session". Yes it's not really ideal, I will try to remove it somehow because it's a little bit out of place there.
Since the active session can only be accessed through that button when the Auto-saved sessions is disabled (from configuration) I need to think really well how a good solution. Certain configuration options can create a lot of problems and that's why there might be some redundancy each and there.

2. Yeah... I kind of wanted to have them better explained but I always had problems with available space. Since I don't know the width of that tab list longer description can not appear complete. If I manage somehow to kill the first tab I will try to provide better description.

3. True... since this is a feature many users expect I should provide a better description of it. Really good idea to have it there. I will try to add some explanations as soon as possible.

------------------------------------------

Thanks for feedback. Import and export functionality will definitely arrive in a future update (as many other features: see https://github.com/ReDEnergy/SessionSync/issues/40 for more info). In the meantime you still have an option: Firefox Bookmark Manager can export/import all your bookmarks (these will contain the saved sessions as well since they actually saved as bookmarks).

Hi, and thanks for feedback.

- Regarding the right click issue... that's definitely really strange. It should not happen like that since I specifically wrote code to block the default right-click behavior. All I can think right know is that another addon might mix things up and and create this issue. If you have time try disabling all your other addons for 1 minute and see if the same it's happening (if it's an addon fault you should be able to spot it pretty easy)

- Configuring saving date format... That's really a good idea and I will have it done ASAP (I would say by the end of this week)

- Relating to the sorting issue... yes definitely makes sense. I will have to figure out a solution for this. From an alphabetical point of view seems that the sorting is correct but certainly not exactly what anyone would normally expect :)

Really great feedback. Thanks again.

Thanks a lot for appreciation.

Regarding importing of session from Session Manager... I looked a bit into it. Sadly, it does not have any export option or at least I can't seem to find one. I also tried to look into the code to maybe find out where exactly and how sessions are stored but the code is not exactly friendly fast browsing. It would definitely be awesome to have an option to import Session Manager session. Not really sure who long it might take me to write the export function until after I take a look over how exactly sessions are saved. If you can know anything about this please contact me directly via email (or on the project's Github page) . I will be happy to try to find a solution for this.
Manual exporting should work... it's not ideal in your case because it means quite a work, certainly not impossible but still... 160 sessions require a lot of clicks :)

Regarding general evaluations and my responses to them. I think that user reviews reflect a lot about the overall quality of the addon. Of course Session Sync is not perfect, there are so many things to improve and missing features. I try to make sure not to leave any possible misleading information unanswered because it might be of help to others users in the feature.

Of course... can't tell that reviews don't matter. I would say that they are the primary reason for me to continue improving this because I realize that there are a few users that understand and care about this too. Good feedback is always welcomed and it certainly hopes a lot to identify what improvements and features I need to add. Yes, It's a little bit hard for me to get why would someone leave a straight bad review for something free but of course you can't ever please everyone.

Lastly, I think that responding to reviews it's a good indication that I take into consideration everything and although I might not have that much time to invest into updating it as fast as possible I still care about it and will continue to do so.

Again, thanks a lot.

Thanks a lot for appreciation. I promise that it will get even even better in the future :)

Later EDIT: Please contact me directly via email (gabrielivanica@gmail.com) with any question you might have regarding the addon. For more direct ways of communication (if needed) I will send you an email after you contact me first.

P.S If the addons shows that "Current Sessions" entry most probably works as it should but might not work as you expect.
------------------------------------------------
I don't really understand why it would not be saved. I mean there's that tab called "History" where you should find the most recent saved sessions.

At this point the auto-saved function creates a new session entry only when the browser starts. Then every 60 seconds (or whatever you set) the same session will be overwritten. If you close the browser during those 60 seconds when you open it again you should find that specific session saved in the History tab.

Only manually saved sessions are stored in the Sessions tab since those are stored as bookmarks while the auto-saved one are stored in internal memory and can will be automatically deleted when the storage (that you also set) is full.

Thanks for the review. Regarding quick-start guide, I sort of have something in the form of those tutorial images that present all the main features of the addon (the Tutorial was suppose to be automatically opened after you installed the addon but you can always re-open it from the top-right addon menu button -> About)
Regarding opening of multiple session at once, sadly this is not possible right now but I do have plans to support with a future update. Sorry for not providing this feature yet.
(For more info regarding feature updates you can visit https://github.com/ReDEnergy/SessionSync/issues/40)