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

Commit

Permalink
Fixed button and title cases for Tags screen
Browse files Browse the repository at this point in the history
no refs.
Set "New tag" button and "Tag settings" to sentence case to be consistent with the rest of the UI. (Previously they were title case.)
  • Loading branch information
peterzimon committed Mar 25, 2019
1 parent 133eb21 commit 5dcf5b9
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions app/components/gh-tag-settings-form.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,9 @@ export default Component.extend({

title: computed('tag.isNew', function () {
if (this.get('tag.isNew')) {
return 'New Tag';
return 'New tag';
} else {
return 'Tag Settings';
return 'Tag settings';
}
}),

Expand Down
2 changes: 1 addition & 1 deletion app/templates/settings/tags-loading.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<header class="view-header">
{{#gh-view-title}}<span>Tags</span>{{/gh-view-title}}
<section class="view-actions">
{{#link-to "settings.tags.new" class="gh-btn gh-btn-green"}}<span>New Tag</span>{{/link-to}}
{{#link-to "settings.tags.new" class="gh-btn gh-btn-green"}}<span>New tag</span>{{/link-to}}
</section>
</header>

Expand Down
2 changes: 1 addition & 1 deletion app/templates/settings/tags.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<header class="view-header">
{{#gh-view-title}}<span>Tags</span>{{/gh-view-title}}
<section class="view-actions">
{{#link-to "settings.tags.new" class="gh-btn gh-btn-green"}}<span>New Tag</span>{{/link-to}}
{{#link-to "settings.tags.new" class="gh-btn gh-btn-green"}}<span>New tag</span>{{/link-to}}
</section>
</header>

Expand Down
2 changes: 1 addition & 1 deletion app/templates/settings/tags/index.hbs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<div class="no-posts-box">
<div class="no-posts">
<h3>You haven't added any tags yet!</h3>
{{#link-to "settings.tags.new"}}<button type="button" class="gh-btn gh-btn-green btn-lg" title="New Tag"><span>Add a tag</span></button>{{/link-to}}
{{#link-to "settings.tags.new"}}<button type="button" class="gh-btn gh-btn-green btn-lg" title="New tag"><span>Add a tag</span></button>{{/link-to}}
</div>
</div>
2 changes: 1 addition & 1 deletion tests/acceptance/settings/tags-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ describe('Acceptance: Settings - Tags', function () {

// it shows selected tag form
expect(find('.tag-settings-pane h4').textContent, 'settings pane title')
.to.equal('Tag Settings');
.to.equal('Tag settings');
expect(find('.tag-settings-pane input[name="name"]').value, 'loads correct tag into form')
.to.equal(tag1.name);

Expand Down
4 changes: 2 additions & 2 deletions tests/integration/components/gh-tag-settings-form-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,10 @@ describe('Integration: Component: gh-tag-settings-form', function () {
await render(hbs`
{{gh-tag-settings-form tag=tag setProperty=(action setProperty)}}
`);
expect(find('.tag-settings-pane h4').textContent, 'existing tag title').to.equal('Tag Settings');
expect(find('.tag-settings-pane h4').textContent, 'existing tag title').to.equal('Tag settings');

this.set('tag.isNew', true);
expect(find('.tag-settings-pane h4').textContent, 'new tag title').to.equal('New Tag');
expect(find('.tag-settings-pane h4').textContent, 'new tag title').to.equal('New tag');
});

it('renders main settings', async function () {
Expand Down

0 comments on commit 5dcf5b9

Please sign in to comment.