Fixes #4848 - Header Without text labels

locales/en-US/server.ftl

@@ -7,8 +7,6 @@ gHomeLink = Home
 gNoShots
     .alt = No shots found
 gScreenshotsDescription = Screenshots made simple. Take, save, and share screenshots without leaving Firefox.
-gSettings = Settings
-gSignIn = Sign In
 
 ## Header
 buttonSettings =
@@ -98,7 +96,6 @@ shotPageAlertErrorUpdatingTitle = Error saving title
 shotPageConfirmDelete = Are you sure you want to delete this shot permanently?
 shotPageShareButton =
     .title = Share
-shotPageCopy = Copy
 shotPageCopyButton =
     .title = Copy image to clipboard
 shotPageCopied = Copied
@@ -135,13 +132,6 @@ shotPageBackToHomeButton =
     .title = Homepage
 shotPageAllShotsButton =
     .title = All Shots
-shotPageAllShots = All Shots
-shotPageDownload = Download
-# Note: Draw text is used on shot page as a verb (action)
-shotPageDraw = Draw
-# Note: Favorite text is used on shot page as a verb (action)
-shotPageFavorite = Favorite
-shotPageDelete = Delete
 shotPageScreenshotsDescription = Screenshots made simple. Take, save, and share screenshots without leaving Firefox.
 shotPageDMCAMessage = This shot is no longer available due to a third party intellectual property claim.
 # Note: { $dmca } is a placeholder for a link to send email (a 'mailto' link)

server/src/delete-shot-button.js

@@ -76,14 +76,10 @@ exports.DeleteShotButton = class DeleteShotButton extends React.Component {
     }
 
     let deleteButtonStyle = null;
-    let deleteText = null;
     if (this.props.isIcon) {
       deleteButtonStyle = `button transparent trash ${deletePanelOpenClass}`;
     } else {
-      deleteButtonStyle = `nav-button transparent icon-trash trash ${deletePanelOpenClass}`;
-      deleteText = <Localized id="shotPageDelete">
-          <span>Delete</span>
-        </Localized>;
+      deleteButtonStyle = `button nav-button transparent ${deletePanelOpenClass}`;
     }
 
     return (
@@ -93,7 +89,7 @@ exports.DeleteShotButton = class DeleteShotButton extends React.Component {
             title="Delete this shot permanently"
             onClick={this.onClickDelete.bind(this)}
             ref={this.trashButtonRef}>
-            { deleteText }
+            <img src={this.props.staticLink("/static/img/icon-trash.svg")} />
           </button>
         </Localized>
         { confirmationPanel }
@@ -107,4 +103,5 @@ exports.DeleteShotButton.propTypes = {
   confirmDeleteHandler: PropTypes.func,
   cancelDeleteHandler: PropTypes.func,
   isIcon: PropTypes.bool,
+  staticLink: PropTypes.func,
 };

server/src/header.js

@@ -21,7 +21,7 @@ exports.Header = function Header(props) {
 
 exports.Header.propTypes = {
   hasLogo: PropTypes.bool,
-  children: PropTypes.node.isRequired,
+  children: PropTypes.node,
   isOwner: PropTypes.bool,
   hasFxa: PropTypes.bool,
   shouldGetFirefox: PropTypes.bool,

server/src/pages/homepage/homepage-header.js

@@ -17,20 +17,20 @@ exports.HomePageHeader = class HomePageHeader extends React.Component {
   renderFxASignIn() {
       return (
         this.props.isFirefox && this.props.isOwner ?
-          <SignInButton isAuthenticated={this.props.hasFxa} initialPage="" /> : null
+          <SignInButton isAuthenticated={this.props.hasFxa} initialPage=""
+            staticLink={this.props.staticLink}/> : null
       );
   }
 
   render() {
     let myShots;
     if (this.props.isOwner) {
       myShots = <Localized id="shotIndexPageMyShotsButton" attrs={{title: true}}>
-        <a className="nav-button icon-shots" title="My Shots" href="/shots" onClick={ this.onClickMyShots.bind(this) } tabIndex="0">
-          <Localized id="gMyShots">
-            <span>My Shots</span>
-          </Localized>
-        </a>
-      </Localized>;
+          <a className="nav-button icon-shots" title="My Shots" href="/shots"
+            onClick={ this.onClickMyShots.bind(this) } tabIndex="0">
+            <img src={this.props.staticLink("/static/img/icon-shots.svg")} />
+          </a>
+        </Localized>;
     }
 
     const signin = this.renderFxASignIn();
@@ -49,4 +49,5 @@ exports.HomePageHeader.propTypes = {
   hasFxa: PropTypes.bool,
   isOwner: PropTypes.bool,
   isFirefox: PropTypes.bool,
+  staticLink: PropTypes.func,
 };

server/src/pages/homepage/view.js

@@ -78,7 +78,7 @@ class Body extends React.Component {
     return (
       <reactruntime.BodyTemplate {...this.props}>
         <HomePageHeader isOwner={this.props.showMyShots} isFirefox={this.props.isFirefox}
-          hasFxa={this.props.hasFxa} />
+          hasFxa={this.props.hasFxa} staticLink={this.props.staticLink}/>
         <div className="banner">
           <div className="banner-image-back" />
           <div className="banner-container">
@@ -168,6 +168,7 @@ Body.propTypes = {
   firefoxVersion: PropTypes.string,
   isFirefox: PropTypes.bool,
   showMyShots: PropTypes.bool,
+  staticLink: PropTypes.func,
 };
 
 exports.HeadFactory = React.createFactory(Head);

server/src/pages/shot/editor.js

@@ -184,7 +184,7 @@ exports.Editor = class Editor extends React.Component {
     const redoButtonState = this.state.canRedo ? "active" : "inactive";
 
     return <div className="editor-header">
-      <div className="shot-main-actions annotation-main-actions">
+      <div className="shot-edit-main-actions annotation-main-actions">
         <div className="annotation-tools">
           <Localized id="annotationCropButton" attrs={{title: true}}>
             <button className={`button transparent crop-button`} id="crop" onClick={this.onClickCrop.bind(this)} title="Crop"></button>
@@ -217,7 +217,7 @@ exports.Editor = class Editor extends React.Component {
           </Localized>
         </div>
       </div>
-      <div className="shot-alt-actions">
+      <div className="shot-edit-alt-actions">
         <Localized id="annotationSaveEditButton" attrs={{title: true}}>
           <button className="button primary save" id="save" onClick={ this.onClickSave.bind(this) } disabled = { this.state.actionsDisabled } title="Save edit">Save</button>
         </Localized>

server/src/pages/shot/shotpage-header.js

@@ -33,9 +33,7 @@ exports.ShotPageHeader = class ShotPageHeader extends React.Component {
     return (
       <Localized id="shotPageAllShotsButton" attrs={{title: true}}>
         <a className="nav-button icon-shots" href="/shots" tabIndex="0" title="All Shots">
-          <Localized id="shotPageAllShots">
-            <span>All Shots</span>
-          </Localized>
+          <img src={this.props.staticLink("/static/img/icon-shots.svg")} />
         </a>
       </Localized>
     );
@@ -55,14 +53,14 @@ exports.ShotPageHeader = class ShotPageHeader extends React.Component {
     const timeDiff = <TimeDiff date={shot.createdDate} />;
     let expirationSubtitle;
     if (this.props.expireTime === null) {
-      expirationSubtitle = <Localized id="shotPageDoesNotExpire"><span>does not expire</span></Localized>;
+      expirationSubtitle = <Localized id="shotPageDoesNotExpire"><span className="expire-info">does not expire</span></Localized>;
     } else {
       const expired = this.props.expireTime < Date.now();
       const expireTimeDiff = <TimeDiff date={this.props.expireTime}/>;
       if (expired) {
-        expirationSubtitle = <Localized id="shotPageTimeExpired" timediff={expireTimeDiff}><span>expired {expireTimeDiff}</span></Localized>;
+        expirationSubtitle = <Localized id="shotPageTimeExpired" timediff={expireTimeDiff}><span className="expire-info">expired {expireTimeDiff}</span></Localized>;
       } else {
-        expirationSubtitle = <Localized id="shotPageTimeExpiresIn" timediff={expireTimeDiff}><span>expires {expireTimeDiff}</span></Localized>;
+        expirationSubtitle = <Localized id="shotPageTimeExpiresIn" timediff={expireTimeDiff}><span className="expire-info">expires {expireTimeDiff}</span></Localized>;
       }
     }
 
@@ -72,7 +70,7 @@ exports.ShotPageHeader = class ShotPageHeader extends React.Component {
           <EditableTitle title={shot.title} isOwner={this.props.isOwner} />
           <div className="shot-subtitle">
             { linkTextShort ? <a className="subtitle-link" rel="noopener noreferrer" href={ shot.url } target="_blank" onClick={ this.onClickOrigUrl.bind(this, "navbar") }>{ linkTextShort }</a> : null }
-            <span className="time-diff">{ timeDiff }</span>
+            <span className="time-diff expire-info">{ timeDiff }</span>
             { expirationSubtitle }
           </div>
         </div>
@@ -93,7 +91,8 @@ exports.ShotPageHeader = class ShotPageHeader extends React.Component {
     return (
       this.props.isOwner ?
         <div className="shot-fxa-signin">
-          <SignInButton isAuthenticated={this.props.isFxaAuthenticated} initialPage={this.props.shot.id} />
+          <SignInButton isAuthenticated={this.props.isFxaAuthenticated} initialPage={this.props.shot.id}
+            staticLink={this.props.staticLink}/>
         </div> : null
     );
   }
@@ -110,8 +109,8 @@ exports.ShotPageHeader = class ShotPageHeader extends React.Component {
         { shotInfo }
         <div className="shot-alt-actions">
         { this.props.children }
-        { signin }
         </div>
+        { signin }
       </Header>
     );
   }
@@ -124,6 +123,7 @@ exports.ShotPageHeader.propTypes = {
   isFxaAuthenticated: PropTypes.bool,
   expireTime: PropTypes.number,
   shouldGetFirefox: PropTypes.bool,
+  staticLink: PropTypes.func,
 };
 
 class EditableTitle extends React.Component {
@@ -161,7 +161,7 @@ class EditableTitle extends React.Component {
       <input ref={(input) => this.textInput = input}
         className="shot-title-input"
         style={{minWidth: this.state.minWidth}}
-        type="text" defaultValue={this.props.title} autoFocus="true"
+        type="text" defaultValue={this.props.title} autoFocus={true}
         onBlur={this.onExit.bind(this)} onKeyUp={this.onKeyUp.bind(this)} onFocus={this.onFocus} />
     </form>;
   }

server/src/pages/shot/view.js

@@ -356,68 +356,63 @@ class Body extends React.Component {
     let favoriteShotButton = null;
     let trashOrFlagButton = null;
     let editButton = null;
-    const highlight = this.state.highlightEditButton
-      ? <div className="edit-highlight"
-          onClick={this.onClickEdit.bind(this)}
-          onMouseOver={this.onMouseOverHighlight.bind(this)}
-          onMouseOut={this.onMouseOutHighlight.bind(this)}></div>
-      : null;
-    const activeFavClass = this.props.expireTime ? "" : "is-fav";
-    const inactive = this.props.isFxaAuthenticated ? "" : "inactive";
-
-    favoriteShotButton = <div className="favorite-shot-button">
-      <Localized id="shotPagefavoriteButton" attrs={{title: true}}>
-        <button className={`nav-button ${inactive}`}
-          disabled={!this.props.isFxaAuthenticated}
-          onClick={this.onClickFavorite.bind(this)}>
-          <span className={`icon-favorite favorite ${activeFavClass}`} ></span>
-          <Localized id="shotPageFavorite">
-            <span className={`favorite-text favorite ${activeFavClass} `}>Favorite</span>
-          </Localized>
-        </button></Localized></div>;
-
-    const downloadButton = <div className="download-shot-button">
-      <Localized id="shotPageDownloadShot" attrs={{title: true}}>
-        <button className="nav-button icon-download" onClick={this.onClickDownload.bind(this)}
-          title="Download the shot image">
-          <Localized id="shotPageDownload">
-            <span>Download</span>
-          </Localized>
-        </button>
-      </Localized></div>;
-
-    const copyButton = <div className="copy-img-button" hidden={!this.state.canCopy}>
-      <Localized id="shotPageCopyButton" attrs={{title: true}}>
-        <button className="nav-button icon-copy transparent copy"
-          title="Copy image to clipboard"
-          onClick={this.onClickCopy.bind(this)}>
-          <Localized id="shotPageCopy">
-            <span>Copy Image</span>
-          </Localized>
-        </button>
-      </Localized>
-    </div>;
+    let downloadButton = null;
+    let copyButton = null;
 
     if (this.props.isOwner) {
+      const highlight = this.state.highlightEditButton
+        ? <div className="edit-highlight"
+            onClick={this.onClickEdit.bind(this)}
+            onMouseOver={this.onMouseOverHighlight.bind(this)}
+            onMouseOut={this.onMouseOutHighlight.bind(this)}></div>
+        : null;
+      const favImgSrc = this.props.expireTime ? this.props.staticLink("/static/img/icon-heart-outline.svg") :
+        this.props.staticLink("/static/img/icon-heart.svg");
+      const inactive = this.props.isFxaAuthenticated ? "" : "inactive";
+
+      favoriteShotButton = <div className="favorite-shot-button">
+        <Localized id="shotPagefavoriteButton" attrs={{title: true}}>
+          <button className={`button transparent nav-button ${inactive}`}
+            disabled={!this.props.isFxaAuthenticated} onClick={this.onClickFavorite.bind(this)}>
+            <img src={favImgSrc} />
+          </button>
+        </Localized></div>;
+
       trashOrFlagButton = <DeleteShotButton
-        clickDeleteHandler={ this.clickDeleteHandler.bind(this) }
-        confirmDeleteHandler={ this.confirmDeleteHandler.bind(this) }
-        cancelDeleteHandler={ this.cancelDeleteHandler.bind(this) } />;
+        clickDeleteHandler={this.clickDeleteHandler.bind(this)}
+        confirmDeleteHandler={this.confirmDeleteHandler.bind(this)}
+        cancelDeleteHandler={this.cancelDeleteHandler.bind(this)}
+        staticLink={this.props.staticLink} />;
 
       editButton = this.props.enableAnnotations ? <div className="edit-shot-button">
         <Localized id="shotPageEditButton" attrs={{title: true}}>
-          <button className="nav-button icon-edit transparent edit"
+          <button className="button transparent nav-button"
             title="Edit this image"
             onClick={this.onClickEdit.bind(this)}
             ref={(edit) => { this.editButton = edit; }}>
-            <Localized id="shotPageDraw">
-              <span>Draw</span>
-            </Localized>
+            <img src={this.props.staticLink("/static/img/icon-pen.svg")} />
           </button>
         </Localized>
         <PromoDialog promoClose={this.promoClose.bind(this)} display={this.state.promoDialog} />
         { highlight }
         </div> : null;
+
+      downloadButton = <div className="download-shot-button">
+        <Localized id="shotPageDownloadShot" attrs={{title: true}}>
+          <button className={`button transparent nav-button`} onClick={this.onClickDownload.bind(this)}
+            title="Download the shot image">
+            <img src={this.props.staticLink("/static/img/icon-download.svg")} />
+          </button>
+        </Localized></div>;
+
+      copyButton = <div className="copy-img-button" hidden={!this.state.canCopy}>
+        <Localized id="shotPageCopyButton" attrs={{title: true}}>
+          <button className="button nav-button transparent copy"
+            title="Copy image to clipboard"
+            onClick={this.onClickCopy.bind(this)}>
+            <img src={this.props.staticLink("/static/img/icon-copy.svg")} />
+          </button>
+        </Localized></div>;
     }
 
     let clip;
@@ -440,7 +435,8 @@ class Body extends React.Component {
       <reactruntime.BodyTemplate {...this.props}>
         <div id="frame" className="inverse-color-scheme full-height column-space">
         <ShotPageHeader isOwner={this.props.isOwner} isFxaAuthenticated={this.props.isFxaAuthenticated}
-          shot={this.props.shot} expireTime={this.props.expireTime} shouldGetFirefox={renderGetFirefox}>
+          shot={this.props.shot} expireTime={this.props.expireTime} shouldGetFirefox={renderGetFirefox}
+          staticLink={this.props.staticLink}>
           { favoriteShotButton }
           { editButton }
           { copyButton }

server/src/pages/shotindex/myshots-header.js

@@ -5,7 +5,8 @@ const { Header } = require("../../header.js");
 
 exports.MyShotsHeader = function MyShotsHeader(props) {
   const signin = props.enableUserSettings && props.hasDeviceId ?
-    <SignInButton isAuthenticated={props.hasFxa} initialPage="shots" /> : null;
+    <SignInButton isAuthenticated={props.hasFxa} initialPage="shots"
+      staticLink={props.staticLink}/> : null;
 
   return (
     <Header hasLogo={true} isOwner={props.hasDeviceId} hasFxa={props.hasFxa}>
@@ -20,4 +21,5 @@ exports.MyShotsHeader.propTypes = {
   hasFxa: PropTypes.bool,
   hasDeviceId: PropTypes.bool,
   enableUserSettings: PropTypes.bool,
+  staticLink: PropTypes.func,
 };

server/src/pages/shotindex/view.js

@@ -41,7 +41,7 @@ class Body extends React.Component {
       <reactruntime.BodyTemplate {...this.props}>
         <div className="column-space full-height" id="shot-index-page">
           <MyShotsHeader hasDeviceId={this.props.hasDeviceId} hasFxa={this.props.hasFxa}
-            enableUserSettings={this.props.enableUserSettings} />
+            enableUserSettings={this.props.enableUserSettings} staticLink={this.props.staticLink}/>
           { this.props.disableSearch ? null : this.renderSearchForm() }
           <div id="shot-index" className="flex-1">
             { this.renderShots() }
@@ -322,7 +322,7 @@ class Card extends React.Component {
         </Localized>;
     } else if (this.props.hasFxa) {
       favoriteIndicator = <Localized id="shotIndexNonFavoriteIcon" attrs={{title: true}}>
-          <div className="non-fav-shot" title=""></div>
+          <div className="indicator non-fav-shot" title=""></div>
         </Localized>;
     }
 
@@ -369,7 +369,8 @@ class Card extends React.Component {
             isIcon = {true}
             clickDeleteHandler={this.clickDeleteHandler.bind(this)}
             confirmDeleteHandler={this.confirmDeleteHandler.bind(this, shot)}
-            cancelDeleteHandler={this.cancelDeleteHandler.bind(this)} />
+            cancelDeleteHandler={this.cancelDeleteHandler.bind(this)}
+            staticLink={this.props.staticLink} />
         </div>
         {favoriteIndicator}
         {syncedShotIndicator}