fix: better naming of language add button

common-voice · Mar 7, 2021 · aa50e64 · aa50e64
1 parent 9ee94a2
commit aa50e64
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/web/css/root.css b/web/css/root.css
@@ -115,7 +115,6 @@ select {
 }
 
 main {
- padding: 1rem 2rem;
  max-width: 85vw;
  margin: 0 auto;
 }

diff --git a/web/src/components/add-language-section.js b/web/src/components/add-language-section.js
@@ -35,7 +35,7 @@ export default function AddLanguage({ allLanguages, onAdd, languages, pendingLan
  labelText="Add a language you want to contribute to"
  onChange={onLanguageSelect} />
  <button disabled={pendingLanguages || !language}
- onClick={onLanguageAdd} className="add-language">Add</button>
+ onClick={onLanguageAdd} className="add-language">Add Language</button>
  </section>
  );
 }
diff --git a/web/src/components/add-language-section.test.js b/web/src/components/add-language-section.test.js
@@ -13,7 +13,7 @@ const allLanguages = [{
 test('should render submit button', () => {
  render(<AddLanguageSection allLanguages={allLanguages}/>);
  expect(screen.getByRole('button')).toBeTruthy();
- expect(screen.getByText('Add')).toBeTruthy();
+ expect(screen.getByText('Add Language')).toBeTruthy();
 });
 
 test('should disable button on pending languages', () => {

diff --git a/web/src/components/header.js b/web/src/components/header.js
@@ -29,7 +29,7 @@ export default function Header() {
 {/*
  <section id="external-links">
  <a target="_blank" rel="noopener noreferrer" href="https://discourse.mozilla.org/tags/c/voice/sentence-collection">Discourse</a>
- <a target="_blank" rel="noopener noreferrer" href="https:/Common-Voice/sentence-collector/issues">Report Bugs (GitHub)</a>
+ <a target="_blank" rel="noopener noreferrer" href="https:/Common-Voice/sentence-collector/issues">Report Bugs</a>
  <a target="_blank" rel="noopener noreferrer" href="https://discourse.mozilla.org/t/sentence-collector-copyright-issues/52767">Report copyright issues</a>
  <a target="_blank" rel="noopener noreferrer" href="https://commonvoice.mozilla.org/privacy">Privacy</a>
  <a target="_blank" rel="noopener noreferrer" href="https://commonvoice.mozilla.org/terms">Terms</a>