Hi,

It seems to work very well. :)

Three things to improve that :
- Use the url icon instead of "tab-new" when actions are updated (
MainWindow::updateActions() )
- Use a foreach loop to open all recent closed tabs and not only the first
when "Open Closed Tabs" action is clicked ( MainView::openClosedTabs() )
- Use Application::instance()->openUrl(url, Rekonq::SettingsOpenTab) to
follow the SettingsOpenTab behaviour instead of create a new tab and load
the url. (( MainView::openClosedTabs() )

You should create a clone of mainline on gitorious and propose your patch
with a merge request.

Best regards.
Yoann Laissus

2010/4/25 Johannes Zellner <[email protected]>

> Hi,
>
> I've attached the changes that introduce a first version, adding actions
> for
> open recently closed tabs.
>
> Even its really basic, feel free to add comments ;-)
>
> Cheers,
> Johannes
>
> _______________________________________________
> rekonq mailing list
> [email protected]
> https://mail.kde.org/mailman/listinfo/rekonq
>
>
_______________________________________________
rekonq mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/rekonq

Reply via email to