| 2015-12-22 18:26:44 |
Olivier Tilloy |
bug |
|
|
added bug |
| 2015-12-22 18:54:19 |
Michael Sheldon |
webbrowser-app (Ubuntu): status |
New |
Confirmed |
|
| 2015-12-29 21:31:13 |
Olivier Tilloy |
description |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png" |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ edit mode in the downloads page should be disabled if the list is empty
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png" |
|
| 2016-01-11 21:45:47 |
Olivier Tilloy |
description |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ edit mode in the downloads page should be disabled if the list is empty
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png" |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ edit mode in the downloads page should be disabled if the list is empty
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png"
∘ on desktop if I uninstall qtdeclarative5-ubuntu-download-manager0.1, the "Downloads" entry is not visible in the drawer menu, as expected, however the Ctrl+J shortcut still opens the downloads view; the shortcut should be enabled conditionally |
|
| 2016-01-11 22:02:48 |
Olivier Tilloy |
branch linked |
|
lp:~osomon/webbrowser-app/address-downloads-issues |
|
| 2016-01-11 22:02:55 |
Olivier Tilloy |
webbrowser-app (Ubuntu): assignee |
Michael Sheldon (michael-sheldon) |
Olivier Tilloy (osomon) |
|
| 2016-01-11 22:02:57 |
Olivier Tilloy |
webbrowser-app (Ubuntu): status |
Confirmed |
In Progress |
|
| 2016-01-12 17:35:16 |
Olivier Tilloy |
description |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ edit mode in the downloads page should be disabled if the list is empty
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png"
∘ on desktop if I uninstall qtdeclarative5-ubuntu-download-manager0.1, the "Downloads" entry is not visible in the drawer menu, as expected, however the Ctrl+J shortcut still opens the downloads view; the shortcut should be enabled conditionally |
The new "download in browser" feature just landed. This works nicely, but there are a few remaining (mostly minor) issues that need to be addressed now. I’m filing this bug to track them all, we may want to break it up in several bugs if/where relevant.
Code issues:
∘ encapsulation broken in DownloadHandler.qml with references to 'downloadsModel'
∘ encapsulation broken in DownloadDelegate.qml with references to 'downloadManager'
∘ missing autopilot tests for file upload, picking webbrowser-app (including multiple files)
∘ showDownloadsPage() doesn’t make much sense on WebViewImpl, should be an internal function
∘ contentHandlerLoader should have "asynchronous: true"
∘ use default PageHeader from UITK instead of custom BrowserPageHeader?
∘ increase test coverage for the DownloadsModel C++ class
∘ for consistency, downloadsContainer should be made a Loader, as done in https://code.launchpad.net/~osomon/webbrowser-app/keyboard-shortcuts-focus-fixes/+merge/280083
∘ in Downloader.qml and Browser.qml, calls to Component.createObject() should pass the properties as second parameter, instead of setting them afterwards
∘ DownloadsModel should probably be made a singleton type, like HistoryModel and BookmarksModel
∘ includes are not ordered alphabetically in downloads-model.cpp
∘ in DownloadsModel::moveToDownloads(), the [if (QFile::exists(destination))] statement is useless, the do…while loop is enough
Functional issues:
∘ the "No Downloads available" message is confusing in picker mode, it would be clearer if it made it explicit that no download matching the expected file type is available
∘ edit mode in the downloads page should be disabled if the list is empty
∘ while in edit mode in the downloads page, the 'delete' icon should probably be disabled if no download is selected (it currently is enabled and exits select mode)
∘ save link from context menu doesn’t work great with links to other pages: no mime type is displayed (and sometimes even no title), and then the page is saved without an html extension, it can be viewed with the document viewer, but this displays the source of the page, it doesn’t render it (we may need special casing for HTML files in the download folder to open them directly in the browser)
∘ (possibly a bug in the UITK?) upload form that accepts only one file, can end up selecting two: select first, then second, then first again, then second again, and both are selected
∘ when downloading an image to the gallery, I’m seeing this message in the log: Failed moving file from "/home/phablet/.cache/com.ubuntu.gallery/HubIncoming/2/capture2.png" to "/home/phablet/Downloads/capture2.png"
∘ on desktop if I uninstall qtdeclarative5-ubuntu-download-manager0.1, the "Downloads" entry is not visible in the drawer menu, as expected, however the Ctrl+J shortcut still opens the downloads view; the shortcut should be enabled conditionally
∘ click/touch events go through the Downloads page header to items below it, so in narrow mode if one clicks/taps in the right place, they can focus the address bar (thus summoning the OSK) or open the drawer menu |
|
| 2016-01-12 21:34:15 |
Olivier Tilloy |
bug task added |
|
canonical-devices-system-image |
|
| 2016-01-12 21:34:25 |
Olivier Tilloy |
canonical-devices-system-image: status |
New |
In Progress |
|
| 2016-01-13 19:46:25 |
Jean-Baptiste Lallement |
canonical-devices-system-image: status |
In Progress |
Fix Committed |
|
| 2016-01-13 19:46:29 |
Jean-Baptiste Lallement |
canonical-devices-system-image: importance |
Undecided |
High |
|
| 2016-01-13 19:46:38 |
Jean-Baptiste Lallement |
canonical-devices-system-image: assignee |
|
Bill Filler (bfiller) |
|
| 2016-01-13 19:46:42 |
Jean-Baptiste Lallement |
canonical-devices-system-image: milestone |
|
ww02-2016 |
|
| 2016-01-13 21:31:56 |
Launchpad Janitor |
webbrowser-app (Ubuntu): status |
In Progress |
Fix Released |
|
| 2016-01-27 18:38:15 |
Pat McGowan |
canonical-devices-system-image: status |
Fix Committed |
Fix Released |
|
