Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 1055198 - [Video] Hide soft home key during full screen video playba... #23500

Closed
wants to merge 1 commit into from

Conversation

russnicoletti
Copy link

...ck

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

}
}


form[role="dialog"][data-type="confirm"] menu button::-moz-focus-inner {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should move this check inside video.css as this will impact other apps such as gallery using confirm.css

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@@ -192,7 +192,7 @@ <h1 id="overlay-title"></h1>
<section>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add class="software-home-button" on menu at Line 187, that will be needed for the 'Add videos to get started' and related overlay shown

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@@ -9,6 +9,7 @@ const LAYOUT_MODE = {
selection: 'layout-selection',
fullscreenPlayer: 'layout-fullscreen-player'
};
var foo = 0;

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems a redundant variable

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@try-server-hook
Copy link

russnicoletti russnicoletti started tests. Results

@mozilla-autolander-deprecated
Copy link
Contributor

This pull request has been closed due to tree stability issues. Please rebase and re-open the pull request if you still need to land this. Ensure the gaia-try run is green before landing. Sorry for any inconvenience.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants