Closed Bug 890764 Opened 11 years ago Closed 11 years ago

Defect - Navigation App Bar not visible at times when loading websites

Categories

(Firefox for Metro Graveyard :: App Bar, defect, P3)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 25

People

(Reporter: kjozwiak, Assigned: jimm)

References

Details

(Whiteboard: feature=defect c=Browser_views u=metro_firefox_user p=1)

Attachments

(2 files)

If a user taps on the white spaces on the awesome screen several times and then continues to open a website under one of the categories (Top Sites, Bookmarks, Recent History), the "Navigation App Bar" will not be visible while the website is being loaded. - Attached a short video as this could be difficult to explain, notice that the second time around the navigation app bar isn't visible when the website is loading Steps to reproduce the issue: 1) Open Firefox Metro 2) Select "About Us" under the bookmarks (notice that the Navigation App bar is visible while loading) 3) Once the website has been loaded, select the "New Tab" overlay 4) On the awesome screen, tap on the white spaces several times (2-3) and then select "About Us" again (notice that the Navigation App bar is not visible while the website is being loaded) Current Behavior: - Navigation App Bar not always appearing when loading websites Expected Behavior: - The Navigation App Bar should always be visible when loading new websites, it should be dismissed once the user either taps on the website Used the following build: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-07-07-03-11-38-mozilla-central/
Blocks: metrov1triage
No longer blocks: metrov1defect&change
confirming - seems to have something to do with flipping back to the already opened tab.
(In reply to Jim Mathies [:jimm] from comment #1) > confirming - seems to have something to do with flipping back to the already > opened tab. Actually, we open the same page ion the second tab, so we are loading the page again.
Blocks: metrov1defect&change
No longer blocks: metrov1triage
Whiteboard: feature=defect c=Browser_views u=metro_firefox_user p=2 status=verified → feature=defect c=Browser_views u=metro_firefox_user p=2
Priority: -- → P3
Attached patch fix (deleted) — Splinter Review
Basically a one line fix with some cleanup. We shouldn't call dismiss from peekTabs which also dismisses the nav bar. The nav bar should always be visible unless the user edge swipes.
Assignee: nobody → jmathies
Attachment #772584 - Flags: review?(tabraldes)
Whiteboard: feature=defect c=Browser_views u=metro_firefox_user p=2 → feature=defect c=Browser_views u=metro_firefox_user p=1
Blocks: metrov1it10
No longer blocks: metrov1defect&change
Status: NEW → ASSIGNED
QA Contact: jbecerra
Comment on attachment 772584 [details] [diff] [review] fix Review of attachment 772584 [details] [diff] [review]: ----------------------------------------------------------------- I'm not actually able to reproduce this issue (even with patch not applied), but the patch seems fine so...
Attachment #772584 - Flags: review?(tabraldes) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
Mozilla/5.0 (Windows NT 6.2; rv:25.0) Gecko/20130715 Firefox/25.0 Verified the fix with the latest Nightly build, using the steps from bug description: the Navigation App Bar remains visible when loading new websites or reloading pages previously opened in other tabs.
Status: RESOLVED → VERIFIED
Went through the following "Defect" for iteration #12 without any issues. Used the following build: http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-08-15-03-02-03-mozilla-central/ - Went through the original test cases in comment #0 without any issues - Opened over several websites through "Top Sites", "Bookmarks" & "Recent History" without any issues - Opened several tabs using the "+" through the Tab App Bar without any issues - Went through all of the above test cases in Filled View without any issues
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: