[PATCH] New feature: closed tabs trash bin as in Opera
Eduardo Robles Elvira
edulix at gmail.com
Mon Nov 12 21:47:49 GMT 2007
Hello!
The new patch is available as always in my svn [1]. The last fixes include
(stright from svn log):
* qHash can give you the same value for two different input values, using
qint64(this) instead
* optimization: using qDeleteAll when cleaning the list of items in
m_lstHistory
* Better looking strings in the config files
* Fixed some flawed logic in slotOpenLastClosedTab
* Some other minor fixes
*Fixed the ugly hack where the undo manager received a KonqMainWindow in the
undo() slot. undo() with no arguments is back!
* There was really no need for tabContainer in ClosedTabItem, so removing it
from there and from everywhere else where it's not needed anymore
Also take into account:
* As you said, indentation, K_GLOBAL_STATIC and that sort of things can be
done later
* I'm still using the m_lock (I haven't renamed it to m_fileLock yet..) as I
explained in a previous email
* I haven't moved ClosedTabItem to libkonq
About not moving ClosedTabITem to libkonq, I think it's best to keep it where
it is because after all it's only used as a pointer (could be anything) from
within libkonq, meanwhile it's used extensively in konqueror.
So we should keep it there for the moment, and maybe if we find a good way to
create a AppDefinedUndoableItem, we could then rethink what to do. But
probably, if we get to that point, the AppDefinedUndoableItem would end up be
defined in.. well, the app ;-). And that's exactly where ClosedTabItem is
right now hehe
Tomorrow is the last chance for this patch to get into kde 4.0.0? If so I
suggest that we put into konqueror svn as it is, tomorrow I don't know if we
will get much time free to hack.
Greetings,
Eduardo Robles Elvira.
---
[1] svn checkout https://forja.rediris.es/svn/csl-konqueror
--
"The reasonable man adapts himself to the world; the unreasonable one
persists in trying to adapt the world to himself. Therefore all progress
depends on the unreasonable man." (George Bernard Shaw)
More information about the kfm-devel
mailing list