This fixes the cluttering of tabs after pinned tabs are unpinned!

Review Request #109854 - Created April 4, 2013 and discarded

Information
mayank jha
rekonq
317798
Reviewers
rekonq
Just changed the position of setTabData(false); in the function unpinTab() to a position similar as in pinTab()
Tests done and it works fine!!

Files


Andrea Diamantini
mayank jha
Andrea Diamantini
mayank jha
Andrea Diamantini
mayank jha
mayank jha
Review request changed

Status: Discarded

Change Summary:

ooh... I finally understood the problem you were referring!
Your code anyway contains anyway some traces from the previous discarded patch. In example you are yet trying to pin JUST the currentIndex() tab, while the feature is intended to work on ALL tabs you right click to.
A proper fix (at least I hope so) for this is to right-calculate the "availableIndex".
commit f619f0769ae049 tries to.
Loading...