After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 725649 - All tabs are initially shown as loading empty pages when loading the session
All tabs are initially shown as loading empty pages when loading the session
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Interface
git master
Other Mac OS
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-03-04 11:51 UTC by Carlos Garcia Campos
Modified: 2014-03-10 10:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ephy-web-view: Remove ephy_web_view_get_title_composite() (3.09 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
committed Details | Review
ephy-web-view: Rename set_loading_title as set_loading_message (7.13 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
committed Details | Review
ephy-web-view: Remove status_message member (2.92 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
committed Details | Review
ephy-web-view: Move get_title_from_address to ephy-embed-utils (3.61 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
committed Details | Review
ephy-web-view: Move embed-title property to EphyEmbed (20.52 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
none Details | Review
Add title parameter to ephy_shell_new_tab_full to create tabs with an initial title (7.17 KB, patch)
2014-03-04 11:52 UTC, Carlos Garcia Campos
none Details | Review
Add title parameter to ephy_shell_new_tab_full to create tabs with an initial title (7.18 KB, patch)
2014-03-04 17:23 UTC, Carlos Garcia Campos
committed Details | Review
ephy-web-view: Move embed-title property to EphyEmbed (20.55 KB, patch)
2014-03-06 13:38 UTC, Carlos Garcia Campos
committed Details | Review

Description Carlos Garcia Campos 2014-03-04 11:51:26 UTC
When not using the delayed requests, ephy at start up shows all the restored tabs loading with the title "Blank Page". Then you can see how while the pages are actually loaded, when they receive the title, tabs change their title one by one. Since page titles are saved in the session, we can use it as initial title of the tab and never shown the Blank Page. The title will be updated anyway when received during the page load.
Comment 1 Carlos Garcia Campos 2014-03-04 11:52:19 UTC
Created attachment 270891 [details] [review]
ephy-web-view: Remove ephy_web_view_get_title_composite()

This is currently only used to set the window title and it's not
actually needed, since we don't show Loading string in title anymore,
only in the status bar. This is consistent with the title box and tabs
title.
Comment 2 Carlos Garcia Campos 2014-03-04 11:52:25 UTC
Created attachment 270892 [details] [review]
ephy-web-view: Rename set_loading_title as set_loading_message

This is no longer used to set a loading title, since it's always called
with a URL or NULL. Also make it static since it's only used by
ephy-web-view.c. Remove the public getter since it's unused too.
Comment 3 Carlos Garcia Campos 2014-03-04 11:52:30 UTC
Created attachment 270893 [details] [review]
ephy-web-view: Remove status_message member

The status message is actually a combination of the loading message and
link message, and both are already members.
Comment 4 Carlos Garcia Campos 2014-03-04 11:52:35 UTC
Created attachment 270894 [details] [review]
ephy-web-view: Move get_title_from_address to ephy-embed-utils
Comment 5 Carlos Garcia Campos 2014-03-04 11:52:40 UTC
Created attachment 270895 [details] [review]
ephy-web-view: Move embed-title property to EphyEmbed

Since it's the embed title what it represents. The web view already has
a title property inherited from WebKitWebView.
Comment 6 Carlos Garcia Campos 2014-03-04 11:52:45 UTC
Created attachment 270896 [details] [review]
Add title parameter to ephy_shell_new_tab_full to create tabs with an initial title

And use it from ephy-session so that when loading the session and not
using delayed requests, you don't see all the tabs initially with "Blank
Page" title.
Comment 7 Carlos Garcia Campos 2014-03-04 17:23:56 UTC
Created attachment 270924 [details] [review]
Add title parameter to ephy_shell_new_tab_full to create tabs with an initial title

And use it from ephy-session so that when loading the session and not
using delayed requests, you don't see all the tabs initially with "Blank
Page" title.
Comment 8 Carlos Garcia Campos 2014-03-06 13:38:25 UTC
Created attachment 271103 [details] [review]
ephy-web-view: Move embed-title property to EphyEmbed

Since it's the embed title what it represents. The web view already has
a title property inherited from WebKitWebView.