diff --git a/devtools/client/aboutdebugging-new/aboutdebugging.css b/devtools/client/aboutdebugging-new/aboutdebugging.css index 2ff450f896f1..c005496329d0 100644 --- a/devtools/client/aboutdebugging-new/aboutdebugging.css +++ b/devtools/client/aboutdebugging-new/aboutdebugging.css @@ -16,6 +16,7 @@ @import "resource://devtools/client/aboutdebugging-new/src/components/RuntimeActions.css"; @import "resource://devtools/client/aboutdebugging-new/src/components/RuntimeInfo.css"; @import "resource://devtools/client/aboutdebugging-new/src/components/connect/ConnectPage.css"; +@import "resource://devtools/client/aboutdebugging-new/src/components/connect/ConnectSection.css"; @import "resource://devtools/client/aboutdebugging-new/src/components/connect/ConnectSteps.css"; @import "resource://devtools/client/aboutdebugging-new/src/components/connect/NetworkLocationsForm.css"; @import "resource://devtools/client/aboutdebugging-new/src/components/connect/NetworkLocationsList.css"; diff --git a/devtools/client/aboutdebugging-new/src/base.css b/devtools/client/aboutdebugging-new/src/base.css index afe5baa90f29..58dd15b4ef43 100644 --- a/devtools/client/aboutdebugging-new/src/base.css +++ b/devtools/client/aboutdebugging-new/src/base.css @@ -50,6 +50,7 @@ --body-20-font-weight: 700; --caption-20-font-size: 13px; --caption-20-font-weight: 400; + --caption-20-color: var(--grey-50); --display-20-font-size: 36px; --display-20-font-weight: 200; --title-20-font-size: 17px; @@ -69,9 +70,10 @@ --message-font-size: 13px; /* Body 10 in Photon - https://design.firefox.com/photon/visuals/typography.html */ --button-font-size: var(--base-font-size); --micro-font-size: 11px; - --monospace-font-family: monospace; + --card-separator-color: var(--grey-20); + /* * Variables particular to about:debugging */ @@ -127,7 +129,6 @@ p, h1 { margin: 0; } - /* * Utils */ @@ -149,6 +150,13 @@ p, h1 { -moz-user-select: none; } +/* Links that need to look like current text */ +.undecorated-link, +.undecorated-link:hover { + text-decoration: none; + color: currentColor; +} + /* * Typography */ @@ -194,9 +202,8 @@ p, h1 { /* Alternative style for a heading (i.e. h1) */ .alt-heading { - font-weight: 300; - font-size: 1.46em; - line-height: 1.2; /* odd value - from common.inc.css */ + font-weight: var(--title-20-font-weight); + font-size: var(--title-20-font-size); margin-block-start: 0; margin-block-end: calc(var(--base-unit) * 4); @@ -254,20 +261,51 @@ Form controls font-size: 1em; } -/* standard, normal button */ -.default-button { +/* Buttons from Photon */ +.default-button, .primary-button { -moz-appearance: none; - color: var(--grey-90); /* Note: this is from Photon Default button */ - background-color: var(--grey-90-a10); /* Note: this is from Photon Default button */ - font-size: var(--button-font-size); /* Note: this is from Photon Default button */ - margin: 0; - height: calc(var(--base-unit) * 8); /* Note: this is from Photon, not common.css */ + height: calc(var(--base-unit) * 8); padding-inline-start: calc(var(--base-unit) * 5); padding-inline-end: calc(var(--base-unit) * 5); border: none; border-radius: calc(var(--base-unit) / 2); + + font-size: var(--button-font-size); +} + +/* Disabled state for buttons from Photon */ +.default-button:disabled, .primary-button:disabled { + opacity: 0.4; +} + +/* Smaller variant size for buttons, from Photon */ +.default-button--micro, .primary-button--micro { + padding-inline-start: calc(2 * var(--base-unit)); + padding-inline-end: calc(2 * var(--base-unit)); + font-size: var(--micro-font-size); + height: calc(var(--base-unit) * 6); +} + +/* Photon button representing a primary action */ +.primary-button { + color: var(--white-100); + background-color: var(--blue-60); +} + +.primary-button:enabled:hover { + background-color: var(--blue-70); +} + +.primary-button:enabled:active { + background-color: var(--blue-80); +} + +/* Photon standard button */ +.default-button { + color: var(--grey-90); /* Note: this is from Photon Default button */ + background-color: var(--grey-90-a10); /* Note: this is from Photon Default button */ } .default-button:enabled:hover { @@ -278,19 +316,7 @@ Form controls background: var(--grey-90-a30); /* Note: this is from Photon Default button */ } -.default-button:disabled { - opacity: 0.4; /* Note: this is from Photon Default button */ -} - -/* smaller size for a default button */ -.default-button--micro { - padding-inline-start: calc(2 * var(--base-unit)); - padding-inline-end: calc(2 * var(--base-unit)); - font-size: var(--micro-font-size); - height: calc(var(--base-unit) * 6); -} - -/* ghost button. icon button with no background from Photon guidelines */ +/* Photon ghost button. Icon button with no background */ .ghost-button { border: none; border-radius: calc(var(--base-unit) / 2); @@ -310,7 +336,7 @@ Form controls background-color: var(--grey-40); } -/* standard inputs */ +/* Standard inputs */ .default-input { line-height: unset; padding: 0 calc(var(--base-unit) * 2); @@ -322,8 +348,8 @@ Form controls background-color: var(--box-background); } +/* Standard checkbox, from Photon */ .default-checkbox { - /* Note: this styles are from Photon, not common.css */ height: calc(var(--base-unit) * 4); margin-inline-end: var(--base-unit); width: calc(var(--base-unit) * 4); @@ -361,18 +387,15 @@ Form controls } /* - * Card style which is used in debug target item and so on. + * Card UI, from Photon */ .card { background-color: var(--white-100); /* from common.inc.css */ border-radius: var(--base-unit); /* from common.inc.css */ box-shadow: 0 1px 4px var(--grey-90-a10); /* from common.inc.css */ - padding-block: calc(var(--base-unit) * 3) calc(var(--base-unit) * 2); - padding-inline: calc(var(--base-unit) * 3) calc(var(--base-unit) * 2); } -.undecorated-link, -.undecorated-link:hover { - text-decoration: none; - color: currentColor; +.card__heading { + font-size: var(--title-20-font-size); /* Note: this is from Photon Title 20 */ + font-weight: var(--title-20-font-weight); /* Note: this is from Photon Title 20 */ } diff --git a/devtools/client/aboutdebugging-new/src/components/App.css b/devtools/client/aboutdebugging-new/src/components/App.css index 7c67c10f91cd..c29d24508637 100644 --- a/devtools/client/aboutdebugging-new/src/components/App.css +++ b/devtools/client/aboutdebugging-new/src/components/App.css @@ -53,8 +53,3 @@ .page { max-width: var(--page-width); } - -.page__section { - margin-block-end: calc(var(--base-unit) * 12); - --section-inline-margin: calc(var(--alt-heading-icon-size) + var(--alt-heading-icon-gap)); -} diff --git a/devtools/client/aboutdebugging-new/src/components/App.js b/devtools/client/aboutdebugging-new/src/components/App.js index 2183ae68f101..6f7f967e7633 100644 --- a/devtools/client/aboutdebugging-new/src/components/App.js +++ b/devtools/client/aboutdebugging-new/src/components/App.js @@ -130,7 +130,7 @@ class App extends PureComponent { return Switch( {}, Route({ - path: "/connect", + path: "/setup", render: () => this.renderConnect(), }), Route({ @@ -142,7 +142,7 @@ class App extends PureComponent { // in this case maybe we'd like to do something else than a redirect. // See: https://bugzilla.mozilla.org/show_bug.cgi?id=1509897 Route({ - render: () => Redirect({ to: "/connect"}), + render: () => Redirect({ to: "/setup"}), }) ); } diff --git a/devtools/client/aboutdebugging-new/src/components/connect/ConnectPage.css b/devtools/client/aboutdebugging-new/src/components/connect/ConnectPage.css index 15afb04e7013..b771b5de4f15 100644 --- a/devtools/client/aboutdebugging-new/src/components/connect/ConnectPage.css +++ b/devtools/client/aboutdebugging-new/src/components/connect/ConnectPage.css @@ -2,10 +2,42 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -.connect-page__usb__toggle-button { - margin-top: calc(var(--base-unit) * 4); -} - .connect-page__disabled-section { color: var(--grey-40); } + +.connect-page__breather { + margin-block-start: calc(var(--base-unit) * 6); +} + +/* + * +--------+----------------------+ + * | USB | |