[Sugar-devel] [PATCH 1/2] Fix for displaying filename in untitled PDFs SL #3597
Simon Schampijer
simon at schampijer.de
Tue May 22 07:33:45 EDT 2012
On 05/21/2012 06:15 PM, Manuel Quiñones wrote:
> This patch moves the line that sets the title to the
> filename of the PDF, so we display "Loading..." until we
> know the title, and if title is None, we show the filename.
>
> Signed-off-by: Manuel Quiñones<manuq at laptop.org>
> ---
> pdfviewer.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/pdfviewer.py b/pdfviewer.py
> index ef6a6ea..cf435d6 100644
> --- a/pdfviewer.py
> +++ b/pdfviewer.py
> @@ -289,8 +289,6 @@ class PDFTabPage(Gtk.HBox):
> # The title may be given from the Journal:
> if title is not None:
> self._browser.props.title = title
> - else:
> - self._browser.props.title = os.path.basename(requested_uri)
>
> self._browser.props.uri = requested_uri
> self._browser.props.load_status = WebKit.LoadStatus.PROVISIONAL
> @@ -335,6 +333,8 @@ class PDFTabPage(Gtk.HBox):
> pdf_title = self._evince_viewer.get_pdf_title()
> if pdf_title is not None:
> self._browser.props.title = pdf_title
> + else:
> + self._browser.props.title = os.path.basename(self._requested_uri)
>
> def _get_path_from_journal(self, journal_uri):
> """Get the system tree URI of the file for the Journal object."""
That one looks good as well, please push.
Now only http://bugs.sugarlabs.org/ticket/3620 is left in this area! :)
Regards,
Simon
More information about the Sugar-devel
mailing list