MozillaWiki

QA/Reading list: Difference between revisions

Page Discussion

QA/Reading list (view source)

Revision as of 07:12, 30 April 2015

20,387 bytes added ,  30 April 2015
m
→‎Bug fix verification: added results
m (→‎Bug fix verification: added results)
 
(39 intermediate revisions by 2 users not shown)
Line 1: Line 1:
= Desktop Reading List =
= Desktop Reading List =
== Ownership ==
== Ownership ==
Developer contacts: Justin Dolske, Blair McBride, Drew Willcoxon
Developer contacts: Blair McBride, Justin Dolske, Drew Willcoxon


QA Lead: [mailto:andrei.vaida@softvision.ro Andrei Vaida], :avaida on IRC
QA Lead: [mailto:andrei.vaida@softvision.ro Andrei Vaida], :avaida on IRC
Line 9: Line 9:
<small>'''<font color="green">[LANDED]</font>'''</small> Nightly 38: first landed, disabled by default
<small>'''<font color="green">[LANDED]</font>'''</small> Nightly 38: first landed, disabled by default


<small>'''<font color="green">[LANDED]</font>'''</small> Aurora 38: landed, disabled by default
<small>'''<font color="green">[LANDED]</font>'''</small> Aurora 38: landed, currently enabled by default


{{ok|}} Beta 38: tbd
<small>'''<font color="green">[LANDED]</font>'''</small> Beta 38: landed, disabled by default


{{ok|}} Release 38: tbd
{{ok|}} Release 38: tbd
<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<big>'''Timeline'''</big>
<div class="mw-collapsible-content">
* '''<small>UPDATE 2015-03-22</small>''': the pref is <code>true</code> by default on 39.0a1.
* '''<small>UPDATE 2015-03-26</small>''': the pref is <code>true</code> by default on 38.0a2.
* '''<small>UPDATE 2015-03-30</small>''': the pref is <code>true</code> by default on 38.0b1.
</div></div>


===Risk analysis===
===Risk analysis===
Line 20: Line 28:
! Event !! Probability !! Impact !! Current situation !! Mitigation steps
! Event !! Probability !! Impact !! Current situation !! Mitigation steps
|-
|-
| Adding a website to Reading List fails to work. || tbd || high || tbd || Periodical regression testing covers this area.
| Adding a website to Reading List fails to work. || probable || high || There were isolated cases where the add action failed. || Periodical regression testing covers this area.
|-
|-
| Opening a website from the Reading List fails to work. || tbd || high || tbd || Periodical regression testing covers this area.
| Opening a website from the Reading List fails to work. || improbable || high || There were no failures reported for this case. || Periodical regression testing covers this area.
|-
|-
| Removing a website from Reading List fails to work. || tbd || medium || tbd || Periodical regression testing covers this area.
| Removing a website from Reading List fails to work. || probable || medium || There were isolated cases where the remove action failed. || Periodical regression testing covers this area.
|-
|-
| The Reading List sidebar toggle button fails to work. || improbable || high || tbd || Periodical regression testing covers this area.
| The Reading List sidebar toggle button fails to work. || improbable || high || There were no failures reported for this case. || Periodical regression testing covers this area.
|-
|-
| Local reading lists are not saved properly and some/all added items are missing. || tbd || high || tbd || Periodical regression testing covers this area.
| Local reading lists are not saved properly and some/all added items are missing. || improbable || high || There were no failures reported for this case. || Periodical regression testing covers this area.
|-
|-
| Reading List items are not synced up with the Firefox Account. || tbd || medium || tbd || Periodical regression testing covers this area.
| Reading List items are not synced up with the Firefox Account. || probable || medium || There were isolated cases where the sync-up action failed. || Periodical regression testing covers this area.
|-
|-
| Reading List items are not synced down from the Firefox Account. ||  tbd || high || tbd || Periodical regression testing covers this area.
| Reading List items are not synced down from the Firefox Account. ||  probable || high || There were isolated cases where the sync-down action failed. || Periodical regression testing covers this area.
|}
|}


Line 57: Line 65:
== Test cases ==
== Test cases ==
===Overview===
===Overview===
* {{done|}} Draft test suite: '''[https://docs.google.com/spreadsheets/d/1PVhqSPcLNvIEY5SooAO81FAxN4j8MFo3Qb4WkZcONfQ/edit here]'''.
* Draft test suites
* {{todo|}} MozTrap test suite '''[[Template:incomplete|here]]'''.
** {{done|}} [https://docs.google.com/spreadsheets/d/1PVhqSPcLNvIEY5SooAO81FAxN4j8MFo3Qb4WkZcONfQ/edit Smoke and Regression Test Suites].
** {{done|}} [https://docs.google.com/spreadsheets/d/1nL4bF2XeKPsTgxhQgoYvOgu1CLWtBPjLZiligd8GJ-M/edit#gid=1837691267 Cross-platform compatibility Test Suite]
* MozTrap test suites
** {{done|}} [https://moztrap.mozilla.org/manage/cases/?filter-suite=881 Smoke Test Suite]
** {{done|}} [https://moztrap.mozilla.org/manage/cases/?filter-suite=882 Regression Test Suite]
** {{done|}} [https://moztrap.mozilla.org/manage/cases/?filter-suite=887 Cross-platform compatibility Test Suite]


===Targeted scenarios===
===Targeted scenarios===
<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<big>'''Testing focused on basic functionality'''</big>
<big>'''Testing focused on basic functionality'''</big>
<div class="mw-collapsible-content">
* Enable Reading List
* Enable Reading List
* Disable reading List
* Disable reading List
Line 71: Line 86:
* Sync down the Reading List from a Firefox Account
* Sync down the Reading List from a Firefox Account
* Sync up local Reading List to a Firefox Account
* Sync up local Reading List to a Firefox Account
</div></div>


<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<big>'''Testing focused on interoperability'''</big>
<big>'''Testing focused on interoperability'''</big>
<div class="mw-collapsible-content">
* Add web page to Reading List from Reader View's sidebar (non-pb & pb)
* Add web page to Reading List from Reader View's sidebar (non-pb & pb)
* Add web page to Reading List from Location Bar and restart Firefox (non-pb & pb)
* Add web page to Reading List from Location Bar and restart Firefox (non-pb & pb)
Line 94: Line 112:
* Open several websites from the Reading List sidebar subsequently, in separate tabs (at least 15)
* Open several websites from the Reading List sidebar subsequently, in separate tabs (at least 15)
* Pin a tab where Reading List sidebar is enabled
* Pin a tab where Reading List sidebar is enabled
</div></div>


<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<big>'''Testing based on recent bugs and regressions'''</big>
<big>'''Testing based on recent bugs and regressions'''</big>
<div class="mw-collapsible-content">
* Enable the Reading List sidebar on Firefox Developer Edition
* Enable the Reading List sidebar on Firefox Developer Edition
* Add web pages with lengthy titles to Reading List
* Add web pages with lengthy titles to Reading List
Line 106: Line 127:
* Check the "Add page to Reading List" button styling on high-contrast themes
* Check the "Add page to Reading List" button styling on high-contrast themes
* Add a web page to Reading List from a private window
* Add a web page to Reading List from a private window
</div></div>


<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<big>'''Testing Reading List with e10s'''</big>
<big>'''Testing Reading List with e10s'''</big>
* Reading List is currently NOT compatible with e10s.
<div class="mw-collapsible-content">
All regular test cases already available in test suites will be executed with e10s enabled.
</div></div>


== Bug work ==
== Bug work ==
Line 134: Line 159:
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144823 Bug 1144823] - reading list sidebar should be sorted by most recently added<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144823 Bug 1144823] - reading list sidebar should be sorted by most recently added<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144382 Bug 1144382] - title text in reading list sidebar item wraps poorly, leaving partial text visible<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144382 Bug 1144382] - title text in reading list sidebar item wraps poorly, leaving partial text visible<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1145372 Bug 1145372] - The add to reading list url bar button is larger than the other url bar buttons<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1145409 Bug 1145409] - Show the site favicon for each reading list item<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1146946 Bug 1146946] - TypeError: this.selectedItem is null from sidebar.js when using the up/down key when the list is empty.<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147444 Bug 1147444] - Improve the transition when deleting an item from the Reading List<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147440 Bug 1147440] - Improve the transition of the Add to Reading List URL Button<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147479 Bug 1147479] - Improve the transition when adding an item from the Reading List<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148364 Bug 1148364] - On changing an article the "Remove from Reading List"-Button flickers<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148462 Bug 1148462] - When "Reading List" is disabled (browser.readinglist.enabled = false) CTRL+ALT+R should not open its sidebar.<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148562 Bug 1148562] - If I right-click the reading list "+" button (in URL bar), the page gets added to my reading list<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148201 Bug 1148201] - Fix desktop reading list client's use of Last-Modified, If-Unmodified-Since, If-Modified-Since headers<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148240 Bug 1148240] - Reading list item is difficult to delete if the vertical scroll bar is visible<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149869 Bug 1149869] - Repeatedly adding and removing an item from reader mode causes duplicate items to appear in the sidebar<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148762 Bug 1148762] - Reading List not full but scrollable for 1px, causes scroll bar to flash when opening reading list sidebar<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149823 Bug 1149823] - Keep reading list sidebar sorted when new items are added via sync<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149023 Bug 1149023] - unable to remove reading list item<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147409 Bug 1147409] - right click on sites in readlist cannot remove it<br/>
<font color="gray">'''[QE-VERIFY-]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1153121 Bug 1153121] - Write local items to the sync log if any logs have "Trace" level<br/>
<font color="gray">'''[QE-VERIFY-]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1153371 Bug 1153371] - Create shared place for readability content CSS styles<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1151823 Bug 1151823] - Reading list item is missing hover effect in "Sidebars" dropdown panel<br/>
<font color="green">'''[QE-VERIFY+]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1152412 Bug 1152412] - Reading list items that are images get stuck in "loading" state<br/>
<font color="gray">'''[QE-VERIFY-]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148701 Bug 1148701] - React to Backoff and Retry-After headers from Reading List storage servers<br/>
</div>
</div>
</div>
</div>
<br/>


<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
Line 147: Line 192:
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133611 Bug 1133611] - [ReadingList] Add promo blurb to ReadingList bookmarks menu<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133611 Bug 1133611] - [ReadingList] Add promo blurb to ReadingList bookmarks menu<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133610 Bug 1133610] - [ReadingList] Add promo blurb to ReadingList sidebar<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133610 Bug 1133610] - [ReadingList] Add promo blurb to ReadingList sidebar<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133485 Bug 1133485] - Hook up "Add to ReadingList" button in desktop ReaderMode<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133485 Bug 1133485] - Hook up "Add to ReadingList" button in desktop ReaderMode
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131461 Bug 1131461] - [ReadingList] Show status text in the URLBar when interacting with the ReadingList button<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131461 Bug 1131461] - [ReadingList] Show status text in the URLBar when interacting with the ReadingList button<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131460 Bug 1131460] - [ReadingList] Add-to-readinglist button in the URLbar should have an animation similar to the bookmarking animation<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131460 Bug 1131460] - [ReadingList] Add-to-readinglist button in the URLbar should have an animation similar to the bookmarking animation<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457] - [ReadingList] Add a button to the URLBar that allows adding the current page to the Reading List<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457] - [ReadingList] Add a button to the URLBar that allows adding the current page to the Reading List
:↳ <small>'''2015-03-18'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123517 Bug 1123517] - [ReadingList] Implement basic sidebar that lists unread ReadingList items
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123517 Bug 1123517] - [ReadingList] Implement basic sidebar that lists unread ReadingList items
:↳ <small>'''2015-02-25'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms; potential issues reported
:↳ <small>'''2015-02-25'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms; potential issues reported
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133662 Bug 1133662] - [ReadingList] Show something useful in the sidebar when the Reading List is empty<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133662 Bug 1133662] - [ReadingList] Show something useful in the sidebar when the Reading List is empty
:↳ <small>'''2015-03-20'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1124153 Bug 1124153] - [ReadingList] Make the list of items in the Reading List sidebar scale well<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1124153 Bug 1124153] - [ReadingList] Make the list of items in the Reading List sidebar scale well<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123525 Bug 1123525] - [ReadingList] Allow deleting items via the Reading List sidebar<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123525 Bug 1123525] - [ReadingList] Allow deleting items via the Reading List sidebar
:↳ <small>'''2015-03-20'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123519 Bug 1123519] - [ReadingList] Allow adding items to the Reading List via the sidebar<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123519 Bug 1123519] - [ReadingList] Allow adding items to the Reading List via the sidebar<br/>
<font color="gray">'''[NEW]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123518 Bug 1123518] - [ReadingList] Add thumbnails/favicons to items in the Reading List sidebar<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1123518 Bug 1123518] - [ReadingList] Add thumbnails/favicons to items in the Reading List sidebar<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1137231 Bug 1137231] - Double Separator In Bookmarks Menu when ReadingList is disabled<br/>
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms; [https://bugzilla.mozilla.org/show_bug.cgi?id=1146348 Bug 1146348] was filed as a follow-up
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1137231 Bug 1137231] - Double Separator In Bookmarks Menu when ReadingList is disabled
:↳ <small>'''2015-03-02'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
:↳ <small>'''2015-03-02'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1124647 Bug 1124647] - [ReadingList] Implement reading list controls in the reader mode control bar<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1124647 Bug 1124647] - [ReadingList] Implement reading list controls in the reader mode control bar
:↳ <small>'''2015-03-09'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
:↳ <small>'''2015-03-09'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133489 Bug 1133489] - Hook up "Open ReadingList" button in desktop ReaderMode<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1133489 Bug 1133489] - Hook up "Open ReadingList" button in desktop ReaderMode<br/>
:↳ <small>'''2015-03-09'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
:↳ <small>'''2015-03-09'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1142217 Bug 1142217] - Hook up "Open ReadingList" button in desktop ReaderMode<br/>
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1142217 Bug 1142217] - Make the Add/Remove to Reading List toggle button of the reader view actually work
:↳ <small>'''2015-03-16'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-16'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144382 Bug 1144382] - title text in reading list sidebar item wraps poorly, leaving partial text visible
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144380 Bug 1144380] - active state on reading list sidebar persists after switching tabs
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144823 Bug 1144823] - reading list sidebar should be sorted by most recently added
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1145372 Bug 1145372] - The add to reading list url bar button is larger than the other url bar buttons
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144680 Bug 1144680] - The Reading List button from the Location Bar is available for in-content pages
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144675 Bug 1144675] - The Reading List button from the Location Bar should have a distinct icon for pages currently in the list
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144377 Bug 1144377] - hover state on reading list sidebar items remains after the mouse leaves the list
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144781 Bug 1144781] - Add to reading list button sometimes doesn't hide when mouseing out from the urlbar
:↳ <small>'''2015-03-23'''</small>: verified fixed on 39.0a1, across platforms, via [https://bugzilla.mozilla.org/show_bug.cgi?id=1144675 Bug 1144675]
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1145911 Bug 1145911] - Enable reading list by default
:↳ <small>'''2015-03-24'''</small>: verified fixed on 39.0a1, across platforms
:↳ <small>'''2015-03-30'''</small>: verified fixed on 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1143388 Bug 1143388] - Loading an item from the readinglist sidebar shouldn't make the user leave the reader mode if it was active
:↳ <small>'''2015-03-30'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1146946 Bug 1146946] - TypeError: this.selectedItem is null from sidebar.js when using the up/down key when the list is empty.
:↳ <small>'''2015-03-30'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1137556 Bug 1137556] - Make the ReadingList sidebar look like other sidebars
:↳ <small>'''2015-03-30'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1146486 Bug 1146486] - The generic reading list thumbnail graphic is stretched too big and on white background
:↳ <small>'''2015-03-30'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1146358 Bug 1146358] - The attempt of adding a page to Reading List from Reader View fails and causes an error
:↳ <small>'''2015-03-30'''</small>: verified fixed on 39.0a1 and 38.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147440 Bug 1147440] - Improve the transition of the Add to Reading List URL Button
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147444 Bug 1147444] - Improve the transition when deleting an item from the Reading List
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147479 Bug 1147479] - Improve the transition when adding an item from the Reading List
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1147889 Bug 1147889] - need transition when switching between light-dark-sepia
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1 and 39.0a2, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1134446 Bug 1134446] - Automatically open the ReadingList sidebar the first time ReaderMode is used
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1 and 39.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149869 Bug 1149869] - Repeatedly adding and removing an item from reader mode causes duplicate items to appear in the sidebar
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1, across platforms
:↳ <small>'''2015-04-10'''</small>: verified fixed on 38.0b3-build1 (20150409144858), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1137089 Bug 1137089] - The sidebar header is white on the developer edition theme
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1 and 39.0a2, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149023 Bug 1149023] - unable to remove reading list item
:↳ <small>'''2015-04-02'''</small>: verified fixed on 40.0a1 and 39.0a2, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148462 Bug 1148462] - When "Reading List" is disabled (browser.readinglist.enabled = false) CTRL+ALT+R should not open its sidebar.
:↳ <small>'''2015-04-06'''</small>: verified fixed on 40.0a1, across platforms
:↳ <small>'''2015-04-07'''</small>: verified fixed on 38.0b2-build1 (20150406174117), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1144774 Bug 1144774] - Add to reading list button is blurry
:↳ <small>'''2015-04-10'''</small>: verified fixed on 38.0b3-build1 (20150409144858), across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148201 Bug 1148201] - Fix desktop reading list client's use of Last-Modified, If-Unmodified-Since, If-Modified-Since headers
:↳ <small>'''2015-04-14'''</small>: verified fixed on the 38 and 39 branches, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148273 Bug 1148273] - Readinglist scheduler doesn't handle FxA error states correctly.
:↳ <small>'''2015-04-14'''</small>: verified fixed on the 38 and 39 branches, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149136 Bug 1149136] - Incorrect hover styling displayed for the article remove button available in the Reading List sidebar
:↳ <small>'''2015-04-16'''</small>: verified fixed on 38.0b4-build1, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1149302 Bug 1149302] - Error when adding an item to reading list
:↳ <small>'''2015-04-16'''</small>: verified fixed on 38.0b4-build1, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1152412 Bug 1152412] - Reading list items that are images get stuck in "loading" state
:↳ <small>'''2015-04-16'''</small>: verified fixed on 40.0a1, across platforms
<font color="green">'''[VERI]'''</font> [https://bugzilla.mozilla.org/show_bug.cgi?id=1148762 Bug 1148762] - Reading List not full but scrollable for 1px, causes scroll bar to flash when opening reading list sidebar
:↳ <small>'''2015-04-29'''</small>: verified fixed on 40.0a1, across platforms
</div>
</div>
</div>
</div>
<br/>
 
<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">
<div class="toccolours mw-collapsible mw-collapsed" style="width:auto">


Line 182: Line 313:
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144668 Bug 1144668] - Clicking the Add to Reading List button from the Location Bar does not trigger the Show Bookmarks icon animation
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144668 Bug 1144668] - Clicking the Add to Reading List button from the Location Bar does not trigger the Show Bookmarks icon animation
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-19'''</small>: marked as <font color="gray"><small>'''DUPE'''</small></font> of [https://bugzilla.mozilla.org/show_bug.cgi?id=1131460 Bug 1131460]
:↳ <small>'''2015-03-19'''</small>: marked as <font color="gray"><small>'''DUPE'''</small></font> of [https://bugzilla.mozilla.org/show_bug.cgi?id=1131460 Bug 1131460] by Tim Nguyen
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144675 Bug 1144675] - The Reading List button from the Location Bar should have a distinct icon for pages currently in the list
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144675 Bug 1144675] - The Reading List button from the Location Bar should have a distinct icon for pages currently in the list
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
Line 193: Line 324:
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144702 Bug 1144702] - The Reading List button from the Location Bar is not adapting to high contrast and Developer Edition themes
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144702 Bug 1144702] - The Reading List button from the Location Bar is not adapting to high contrast and Developer Edition themes
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-23'''</small>: closed as <font color="gray"><small>'''DUPE'''</small></font> of [https://bugzilla.mozilla.org/show_bug.cgi?id=1134658 Bug 1134658], by Gavin Sharp
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144716 Bug 1144716] - Toggling the Reading List button from the Location Bar of private windows throws errors in the Browser Console
[https://bugzilla.mozilla.org/show_bug.cgi?id=1144716 Bug 1144716] - Toggling the Reading List button from the Location Bar of private windows throws errors in the Browser Console
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-18'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1131457 Bug 1131457]
:↳ <small>'''2015-03-23'''</small>: closed as <font color="gray">'''<small>WFM</small>'''</font>, as the error is no longer reproducible on 39.0a1
[https://bugzilla.mozilla.org/show_bug.cgi?id=1145565 Bug 1145565] - The "Show Reading List Sidebar" button from the bookmarks menu should be visible at all times
:↳ <small>'''2015-03-20'''</small>: found while running Reading List test cases
[https://bugzilla.mozilla.org/show_bug.cgi?id=1145609 Bug 1145609] - The Reading List item removal button should have the same tooltip as the one from the Location Bar
:↳ <small>'''2015-03-20'''</small>: found while verifying the fix pushed for [https://bugzilla.mozilla.org/show_bug.cgi?id=1123525 Bug 1123525]
[https://bugzilla.mozilla.org/show_bug.cgi?id=1149136 Bug 1149136] - Incorrect hover styling displayed for the article remove button available in the Reading List sidebar
:↳ <small>'''2015-03-30'''</small>: found while signing off the feature for the merge to beta
[https://bugzilla.mozilla.org/show_bug.cgi?id=1149144 Bug 1149144] - Crash-report page thumbnail displays a never-ending loading circle in Reading List sidebar
:↳ <small>'''2015-03-30'''</small>: found while signing off the feature for the merge to beta
[https://bugzilla.mozilla.org/show_bug.cgi?id=1149902 Bug 1149902] - The Reading List's sidebar is sometimes enabled automatically in newly opened private windows
:↳ <small>'''2015-04-01'''</small>: found while verifying build1 of 38.0b1-build1
[https://bugzilla.mozilla.org/show_bug.cgi?id=1150369 Bug 1150369] - Intermittently, syncing Reading List items fails with an error thrown by Sync.jsm:431
:↳ <small>'''2015-04-02'''</small>: found while verifying build1 of 38.0b1-build1
:↳ <small>'''2015-04-02'''</small>: marked as <font color="gray">'''<small>DUPE</small>'''</font> of [https://bugzilla.mozilla.org/show_bug.cgi?id=1149105 Bug 1149105] by Mark Hammond
[https://bugzilla.mozilla.org/show_bug.cgi?id=1150388 Bug 1150388] - Some items from Reading List are doubled after the automated syncing with Firefox Account
:↳ <small>'''2015-04-02'''</small>: found while verifying build1 of 38.0b1-build1
[https://bugzilla.mozilla.org/show_bug.cgi?id=1151845 Bug 1151845] - Thumbnail and name for a previous played youtube video are added in Reading list
:↳ <small>'''2015-04-07'''</small>: found during the verification of 38.0b2-build1, by Petruta Rasa
[https://bugzilla.mozilla.org/show_bug.cgi?id=1151871 Bug 1151871] - Links with the same domain can not be added to Reading List
:↳ <small>'''2015-04-07'''</small>: found during the verification of 38.0b2-build1, by Petruta Rasa
[https://bugzilla.mozilla.org/show_bug.cgi?id=1151823 Bug 1151823] - Reading list item is missing hover effect in "Sidebars" dropdown panel
:↳ <small>'''2015-04-07'''</small>: found during the verification of 38.0b2-build1, by Catalin Varga
[https://bugzilla.mozilla.org/show_bug.cgi?id=1152192 Bug 1152192] - Repeatedly adding and removing an item from Reader View causes duplicates in Reading List's sidebar
:↳ <small>'''2015-04-08'''</small>: filed as a follow-up issue for [https://bugzilla.mozilla.org/show_bug.cgi?id=1149869 Bug 1149869]
[https://bugzilla.mozilla.org/show_bug.cgi?id=1152307 Bug 1152307] - Reading list is not synced
:↳ <small>'''2015-04-08'''</small>: found by Mihaela Velimiroviciu while testing Reading List with e10s enabled on 40.0a1
</div>
</div>
</div>
</div>
<br/>


== Sign off ==
== Sign off ==
Line 210: Line 367:
** Manual testing covered Windows 7 (x64), Windows 8.1 (x86), Ubuntu 14.04 (x64) and Mac OS X 10.9.5.
** Manual testing covered Windows 7 (x64), Windows 8.1 (x86), Ubuntu 14.04 (x64) and Mac OS X 10.9.5.


'''Merge to Beta Sign-off''' ''(date)''
'''Merge to Beta Sign-off'''  
* {{ok|}} Beta 38 ''(date)''.
* {{done|}} Beta 38 ''(2015-03-30)''.
** There were no blockers found while testing this feature. Detailed test results are available in [https://etherpad.mozilla.org/Fx38-ReadingList this etherpad].
** Manual testing covered Windows 7 (x64), Windows 8.1  (x64), Windows XP (x64), Ubuntu 14.04 (x86) and Mac OS X 10.9.5.


'''Merge to Release Sign-off''' ''(date)''
'''Merge to Release Sign-off'''
* {{ok|}} RC 38.0 ''(date)''.
* {{ok|}} RC 38.0 ''(date)''.
Andrei.vaida
Confirmed users
1,595

edits

Retrieved from "https://wiki.mozilla.org/Special:MobileDiff/1063039...1071279"