Deleting a bookmark from the new tab view doesn’t display the next one in the list
Bug #1442190 reported by
Olivier Tilloy
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
webbrowser-app |
Fix Released
|
Medium
|
Riccardo Padovani | ||
webbrowser-app (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
If a user has more than 5 bookmarks and opens a new tab, the first 5 bookmarks are displayed, followed by a "See More" button.
Swiping one of the visible bookmarks to the right to remove it correctly deletes it, but the view isn’t refilled with the next bookmark that was previously hidden, it remains with 4 bookmarks, even though there are more to show.
Related branches
lp:~rpadovani/webbrowser-app/newTabRefactoring
- PS Jenkins bot: Needs Fixing (continuous-integration)
- Olivier Tilloy: Approve
-
Diff: 773 lines (+351/-279)6 files modifiedsrc/app/webbrowser/Browser.qml (+59/-59)
src/app/webbrowser/NewTabView.qml (+200/-126)
src/app/webbrowser/UrlsList.qml (+72/-93)
src/app/webbrowser/bookmarks-model.cpp (+3/-1)
src/app/webbrowser/bookmarks-model.h (+2/-0)
tests/unittests/bookmarks-model/tst_BookmarksModelTests.cpp (+15/-0)
Changed in webbrowser-app: | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in webbrowser-app: | |
assignee: | nobody → Riccardo Padovani (rpadovani) |
status: | Triaged → In Progress |
tags: | added: newtab |
Changed in webbrowser-app: | |
status: | In Progress → Fix Released |
To post a comment you must log in.
This bug was fixed in the package webbrowser-app - 0.23+15. 10.20150602- 0ubuntu1
--------------- 10.20150602- 0ubuntu1) wily; urgency=medium
webbrowser-app (0.23+15.
[ CI Train Bot ]
* New rebuild forced.
[ Olivier Tilloy ] qmlplugin to 1.6 to fix unit
* Actually clear the network cache by deleting the correct set of
files in the correct directory. (LP: #1459956)
* Bump build dependency on liboxideqt-
tests.
* Do not cache favicons on disk when browsing in private mode. (LP:
#1458963)
* Do not try to remove a file that doesn’t exist.
* Pass plain strings to the worker script instead of RegExps. (LP:
#1445673)
* Remove the upstreamcomponents folder, and use components from the
UITK instead. Add autopilot tests for the new tab view.
* Update translation template.
* Updated icon. (LP: #1457424)
[ Riccardo Padovani ]
* New tab view refactoring. (LP: #1371248, #1444023, #1351157,
#1389605, #1442190)
* New tab view refactoring. (LP: #1371248, #1444023, #1351157,
#1389605, #1442190)
-- CI Train Bot <email address hidden> Tue, 02 Jun 2015 14:26:50 +0000