[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-09 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #17 from avlas  ---
(In reply to Stefano Crocco from comment #16)
> I fixed the issue with session restoring not working when preloading is
> enabled. I also made a change which makes Konqueror work more like Dolphin
> in this regard: now, when launching Konqueror with one or more URLs, it will
> both restore the previous state and open the given URLs. Previously, in this
> situation, the saved state was ignored (my reasoning was that if the user
> asked for a specific URL, he didn't want others).

Excellent, it works great! And the new implementation fits with that of other
browsers :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-09 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #16 from Stefano Crocco  ---
I fixed the issue with session restoring not working when preloading is
enabled. I also made a change which makes Konqueror work more like Dolphin in
this regard: now, when launching Konqueror with one or more URLs, it will both
restore the previous state and open the given URLs. Previously, in this
situation, the saved state was ignored (my reasoning was that if the user asked
for a specific URL, he didn't want others).

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-08 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #15 from Stefano Crocco  ---
> Well, actually restore tabs functionality does not work at all if keeping a
> preloaded instance is active, meaning that the link is opened but previous
> tabs are not

I'll look into this

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-08 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #14 from avlas  ---
(In reply to avlas from comment #13)
> I removed konquerorrc from .config (actually just moved it out), which is
> the most similar thing to start from scratch I could imagine. All my
> settings were gone with this. I realized of two things:
> 
> 1. Restore tabs setting does not initialize unless you have had at least 2
> tabs open since you first activated the setting. once you have had 2 tabs
> open, then restore tabs works for any number of tabs, including just 1.
> 
> 2. I always reproduce the issue I mentioned, I can never get external links
> to open neither as tabs (as expected if you have enabled that in settings)
> nor in a second window (as it happens to you). I wonder if this is related
> to the setting: keeping a preloaded instance, which I disabled.
> 
> I will try next with that setting active

Well, actually restore tabs functionality does not work at all if keeping a
preloaded instance is active, meaning that the link is opened but previous tabs
are not

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-08 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #13 from avlas  ---
I removed konquerorrc from .config (actually just moved it out), which is the
most similar thing to start from scratch I could imagine. All my settings were
gone with this. I realized of two things:

1. Restore tabs setting does not initialize unless you have had at least 2 tabs
open since you first activated the setting. once you have had 2 tabs open, then
restore tabs works for any number of tabs, including just 1.

2. I always reproduce the issue I mentioned, I can never get external links to
open neither as tabs (as expected if you have enabled that in settings) nor in
a second window (as it happens to you). I wonder if this is related to the
setting: keeping a preloaded instance, which I disabled.

I will try next with that setting active

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-05 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #12 from avlas  ---
(In reply to Stefano Crocco from comment #11)
> (In reply to avlas from comment #10)
> > (In reply to Stefano Crocco from comment #9)
> > > (In reply to avlas from comment #8)
> > > > (In reply to Stefano Crocco from comment #7)
> > > > I guess that to reproduce the issue you probably need to:
> > > > 
> > > > 1. check the option to restore tabs in settings
> > > > 2. open some tabs
> > > > 3. close konqueror
> > > > 4. click on the email link
> > > > 
> > > > Could you please try to reproduce following these steps?
> > > 
> > > That's what I already tried. I checked again, with both my current user 
> > > and
> > > a new user and the results are the same I already described: a new window
> > > with only the page corresponding to the link. I'll look again into this
> > > issue during the weekend, but I really can't think of something leading to
> > > the behavior you're describing.
> > 
> > I see. I think you may also need to check the setting to open links in new
> > tabs instead of in new windows. Can you try?
> 
> I always have it enabled, so that's not the problem.
> 
> Could you please try with a new user?

Mmm, but that's also unexpected, isn't it? shouldn't the link then open in a
new tab instead of a new window?

I will try to test from scratch and let you know about the result

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-01 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #11 from Stefano Crocco  ---
(In reply to avlas from comment #10)
> (In reply to Stefano Crocco from comment #9)
> > (In reply to avlas from comment #8)
> > > (In reply to Stefano Crocco from comment #7)
> > > I guess that to reproduce the issue you probably need to:
> > > 
> > > 1. check the option to restore tabs in settings
> > > 2. open some tabs
> > > 3. close konqueror
> > > 4. click on the email link
> > > 
> > > Could you please try to reproduce following these steps?
> > 
> > That's what I already tried. I checked again, with both my current user and
> > a new user and the results are the same I already described: a new window
> > with only the page corresponding to the link. I'll look again into this
> > issue during the weekend, but I really can't think of something leading to
> > the behavior you're describing.
> 
> I see. I think you may also need to check the setting to open links in new
> tabs instead of in new windows. Can you try?

I always have it enabled, so that's not the problem.

Could you please try with a new user?

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-06-01 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #10 from avlas  ---
(In reply to Stefano Crocco from comment #9)
> (In reply to avlas from comment #8)
> > (In reply to Stefano Crocco from comment #7)
> > I guess that to reproduce the issue you probably need to:
> > 
> > 1. check the option to restore tabs in settings
> > 2. open some tabs
> > 3. close konqueror
> > 4. click on the email link
> > 
> > Could you please try to reproduce following these steps?
> 
> That's what I already tried. I checked again, with both my current user and
> a new user and the results are the same I already described: a new window
> with only the page corresponding to the link. I'll look again into this
> issue during the weekend, but I really can't think of something leading to
> the behavior you're describing.

I see. I think you may also need to check the setting to open links in new tabs
instead of in new windows. Can you try?

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-31 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #9 from Stefano Crocco  ---
(In reply to avlas from comment #8)
> (In reply to Stefano Crocco from comment #7)
> I guess that to reproduce the issue you probably need to:
> 
> 1. check the option to restore tabs in settings
> 2. open some tabs
> 3. close konqueror
> 4. click on the email link
> 
> Could you please try to reproduce following these steps?

That's what I already tried. I checked again, with both my current user and a
new user and the results are the same I already described: a new window with
only the page corresponding to the link. I'll look again into this issue during
the weekend, but I really can't think of something leading to the behavior
you're describing.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-31 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #8 from avlas  ---
(In reply to Stefano Crocco from comment #7)
> (In reply to avlas from comment #6)
> > Hi Stefano, there is one issue with this new feature (that otherwise is
> > working great!), which is when opening a link while Konqueror is closed.
> > 
> > The expected behavior would be to load all previous tabs and the new link in
> > an additional tab.
> > 
> > The observed behavior is that only the previous tabs are opened, so I need
> > to open the link again to load the new tab.
> 
> I can't reproduce this behavior. If I click on a link, for example in an
> e-mail, Konqueror opens with a single tab displaying that link, which is the
> opposite of what you are experiencing. Are you doing something specific to
> open the link? I tried by clicking a link in an e-mail.

I guess that to reproduce the issue you probably need to:

1. check the option to restore tabs in settings
2. open some tabs
3. close konqueror
4. click on the email link

Could you please try to reproduce following these steps?

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-31 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #7 from Stefano Crocco  ---
(In reply to avlas from comment #6)
> Hi Stefano, there is one issue with this new feature (that otherwise is
> working great!), which is when opening a link while Konqueror is closed.
> 
> The expected behavior would be to load all previous tabs and the new link in
> an additional tab.
> 
> The observed behavior is that only the previous tabs are opened, so I need
> to open the link again to load the new tab.

I can't reproduce this behavior. If I click on a link, for example in an
e-mail, Konqueror opens with a single tab displaying that link, which is the
opposite of what you are experiencing. Are you doing something specific to open
the link? I tried by clicking a link in an e-mail.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-31 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #6 from avlas  ---
Hi Stefano, there is one issue with this new feature (that otherwise is working
great!), which is when opening a link while Konqueror is closed.

The expected behavior would be to load all previous tabs and the new link in an
additional tab.

The observed behavior is that only the previous tabs are opened, so I need to
open the link again to load the new tab.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-07 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #5 from avlas  ---
Awesome!

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-07 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #4 from Stefano Crocco  ---
Note that the option is disabled by default. You must enable it in the
"General" page of the settings dialog.

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-05-07 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

Stefano Crocco  changed:

   What|Removed |Added

 Resolution|--- |FIXED
  Latest Commit||https://invent.kde.org/netw
   ||ork/konqueror/commit/6393fa
   ||fc2633ef2a4112b10abcd841025
   ||29bd587
 Status|ASSIGNED|RESOLVED

--- Comment #3 from Stefano Crocco  ---
Git commit 6393fafc2633ef2a4112b10abcd84102529bd587 by Stefano Crocco.
Committed on 07/05/2023 at 09:49.
Pushed by stefanocrocco into branch 'master'.

Add option to restore last window when starting Konqueror again

M  +10   -1settings/konqhtml/generalopts.cpp
M  +2-0settings/konqhtml/generalopts.h
M  +10   -3src/konqapplication.cpp
M  +13   -0src/konqmainwindow.h
M  +34   -0src/konqsessionmanager.cpp
M  +17   -0src/konqsessionmanager.h

https://invent.kde.org/network/konqueror/commit/6393fafc2633ef2a4112b10abcd84102529bd587

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-04-30 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=421245

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #2 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/network/konqueror/-/merge_requests/188

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-04-02 Thread Stefano Crocco
https://bugs.kde.org/show_bug.cgi?id=421245

Stefano Crocco  changed:

   What|Removed |Added

 Ever confirmed|0   |1
   Severity|normal  |wishlist
 CC||stefano.cro...@alice.it
 Status|REPORTED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.

[konqueror] [Bug 421245] Add save tabs feature (like in Dolphin)

2023-03-27 Thread avlas
https://bugs.kde.org/show_bug.cgi?id=421245

--- Comment #1 from avlas  ---
Still an issue. I recently created a script to save current session each time
there is a change but having this internally in konqueror as it is in dolphin
would be much better

-- 
You are receiving this mail because:
You are watching all bug changes.