From a9a87d948c456e1e2454faee9f0dc2553a367272 Mon Sep 17 00:00:00 2001
From: Randall Morey <randall@randallmorey.com>
Date: Thu, 4 Jun 2020 12:23:06 -0400
Subject: [PATCH] =?UTF-8?q?refactor:=20=F0=9F=92=A1=20rename=20to=20sideba?=
 =?UTF-8?q?r=20nav;=20tweak=20test?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../rose/nav/{primary.js => sidebar.js}           |  2 +-
 .../{primary.stories.mdx => sidebar.stories.mdx}  | 14 +++++++-------
 .../addon/styles/rose/components/nav/_index.scss  |  2 +-
 .../nav/{_primary.scss => _sidebar.scss}          |  2 +-
 .../rose/nav/{primary.hbs => sidebar.hbs}         |  2 +-
 addons/rose/app/components/rose/nav/primary.js    |  1 -
 addons/rose/app/components/rose/nav/sidebar.js    |  1 +
 addons/rose/tests/dummy/app/templates/index.hbs   |  4 ++--
 .../rose/nav/{primary-test.js => sidebar-test.js} | 15 +++++++--------
 ui/core/app/styles/app.scss                       |  2 +-
 ui/core/app/templates/application.hbs             |  4 ++--
 11 files changed, 24 insertions(+), 25 deletions(-)
 rename addons/rose/addon/components/rose/nav/{primary.js => sidebar.js} (79%)
 rename addons/rose/addon/components/rose/nav/{primary.stories.mdx => sidebar.stories.mdx} (74%)
 rename addons/rose/addon/styles/rose/components/nav/{_primary.scss => _sidebar.scss} (97%)
 rename addons/rose/addon/templates/components/rose/nav/{primary.hbs => sidebar.hbs} (91%)
 delete mode 100644 addons/rose/app/components/rose/nav/primary.js
 create mode 100644 addons/rose/app/components/rose/nav/sidebar.js
 rename addons/rose/tests/integration/components/rose/nav/{primary-test.js => sidebar-test.js} (71%)

diff --git a/addons/rose/addon/components/rose/nav/primary.js b/addons/rose/addon/components/rose/nav/sidebar.js
similarity index 79%
rename from addons/rose/addon/components/rose/nav/primary.js
rename to addons/rose/addon/components/rose/nav/sidebar.js
index 47266ffef..40b8900e4 100644
--- a/addons/rose/addon/components/rose/nav/primary.js
+++ b/addons/rose/addon/components/rose/nav/sidebar.js
@@ -1,5 +1,5 @@
 import Component from '@ember/component';
-import layout from '../../../templates/components/rose/nav/primary';
+import layout from '../../../templates/components/rose/nav/sidebar';
 import ComponentAutoId from '../../../mixins/component-auto-id';
 
 /**
diff --git a/addons/rose/addon/components/rose/nav/primary.stories.mdx b/addons/rose/addon/components/rose/nav/sidebar.stories.mdx
similarity index 74%
rename from addons/rose/addon/components/rose/nav/primary.stories.mdx
rename to addons/rose/addon/components/rose/nav/sidebar.stories.mdx
index 36e0e4d4c..353cf0e86 100644
--- a/addons/rose/addon/components/rose/nav/primary.stories.mdx
+++ b/addons/rose/addon/components/rose/nav/sidebar.stories.mdx
@@ -3,19 +3,19 @@ import { hbs } from 'ember-cli-htmlbars';
 import { action } from '@storybook/addon-actions';
 import { text, boolean, select } from '@storybook/addon-knobs';
 
-<Meta title="Rose|Nav/Primary" component="Nav/Primary" />
+<Meta title="Rose|Nav/Sidebar" component="Nav/Sidebar" />
 
-# Primary Navigation
+# Sidebar Navigation
 
 A simple navigation component intended for use in the sidebar layout region.
 
 <Preview>
-  <Story name="Basic Primary Nav" height="8rem">{{
+  <Story name="Basic Sidebar Nav" height="8rem">{{
     template: hbs`
-      <Rose::Nav::Primary @title={{title}} as |nav|>
+      <Rose::Nav::Sidebar @title={{title}} as |nav|>
         <nav.link @route="index" class="active">Item Name</nav.link>
         <nav.link @route="index">Item Name</nav.link>
-      </Rose::Nav::Primary>
+      </Rose::Nav::Sidebar>
     `,
     context: {
       title: text('title', 'Title'),
@@ -27,8 +27,8 @@ _Note_:  do not assign the `active` class directly.  This is done by the link
 component automatically.  We use it in this story for illustrative purposes.
 
 ```htmlbars
-<Rose::Nav::Primary @title="Title" as |nav|>
+<Rose::Nav::Sidebar @title="Title" as |nav|>
   <nav.link @route="index">Item Name</nav.link>
   <nav.link @route="about">Item Name</nav.link>
-</Rose::Nav::Primary>
+</Rose::Nav::Sidebar>
 ```
diff --git a/addons/rose/addon/styles/rose/components/nav/_index.scss b/addons/rose/addon/styles/rose/components/nav/_index.scss
index 77d16beb5..0abffa8bd 100644
--- a/addons/rose/addon/styles/rose/components/nav/_index.scss
+++ b/addons/rose/addon/styles/rose/components/nav/_index.scss
@@ -1 +1 @@
-@import "primary";
+@import "sidebar";
diff --git a/addons/rose/addon/styles/rose/components/nav/_primary.scss b/addons/rose/addon/styles/rose/components/nav/_sidebar.scss
similarity index 97%
rename from addons/rose/addon/styles/rose/components/nav/_primary.scss
rename to addons/rose/addon/styles/rose/components/nav/_sidebar.scss
index e91f6de8f..7dae497d6 100644
--- a/addons/rose/addon/styles/rose/components/nav/_primary.scss
+++ b/addons/rose/addon/styles/rose/components/nav/_sidebar.scss
@@ -1,7 +1,7 @@
 @use '../../variables/sizing';
 @use '../../utilities/type';
 
-.rose-nav-primary {
+.rose-nav-sidebar {
   $threePX: sizing.rems(xxxs) + sizing.rems(xxxxs);
 
   .rose-nav-title {
diff --git a/addons/rose/addon/templates/components/rose/nav/primary.hbs b/addons/rose/addon/templates/components/rose/nav/sidebar.hbs
similarity index 91%
rename from addons/rose/addon/templates/components/rose/nav/primary.hbs
rename to addons/rose/addon/templates/components/rose/nav/sidebar.hbs
index dd0406587..40e3e9eaf 100644
--- a/addons/rose/addon/templates/components/rose/nav/primary.hbs
+++ b/addons/rose/addon/templates/components/rose/nav/sidebar.hbs
@@ -1,6 +1,6 @@
 <nav
   ...attributes
-  class="rose-nav-primary"
+  class="rose-nav-sidebar"
   id={{this.id}}
   aria-labelledby={{if @title (concat "title-" this.id)}}>
 
diff --git a/addons/rose/app/components/rose/nav/primary.js b/addons/rose/app/components/rose/nav/primary.js
deleted file mode 100644
index adaf57f51..000000000
--- a/addons/rose/app/components/rose/nav/primary.js
+++ /dev/null
@@ -1 +0,0 @@
-export { default } from 'rose/components/rose/nav/primary';
\ No newline at end of file
diff --git a/addons/rose/app/components/rose/nav/sidebar.js b/addons/rose/app/components/rose/nav/sidebar.js
new file mode 100644
index 000000000..c8316dbfa
--- /dev/null
+++ b/addons/rose/app/components/rose/nav/sidebar.js
@@ -0,0 +1 @@
+export { default } from 'rose/components/rose/nav/sidebar';
diff --git a/addons/rose/tests/dummy/app/templates/index.hbs b/addons/rose/tests/dummy/app/templates/index.hbs
index 0eb142a9b..a58b48caf 100644
--- a/addons/rose/tests/dummy/app/templates/index.hbs
+++ b/addons/rose/tests/dummy/app/templates/index.hbs
@@ -31,12 +31,12 @@
   </page.header>
 
   <page.sidebar>
-    <Rose::Nav::Primary @title="Title" as |nav|>
+    <Rose::Nav::Sidebar @title="Title" as |nav|>
       <nav.link @route="index">Item Name</nav.link>
       <nav.link @route="about">Item Name</nav.link>
       <nav.link @route="about">Item Name</nav.link>
       <nav.link @route="about">Item Name</nav.link>
-    </Rose::Nav::Primary>
+    </Rose::Nav::Sidebar>
   </page.sidebar>
 
   <page.main>
diff --git a/addons/rose/tests/integration/components/rose/nav/primary-test.js b/addons/rose/tests/integration/components/rose/nav/sidebar-test.js
similarity index 71%
rename from addons/rose/tests/integration/components/rose/nav/primary-test.js
rename to addons/rose/tests/integration/components/rose/nav/sidebar-test.js
index 20229e9e3..dfda90191 100644
--- a/addons/rose/tests/integration/components/rose/nav/primary-test.js
+++ b/addons/rose/tests/integration/components/rose/nav/sidebar-test.js
@@ -3,15 +3,15 @@ import { setupRenderingTest } from 'ember-qunit';
 import { render, find } from '@ember/test-helpers';
 import hbs from 'htmlbars-inline-precompile';
 
-module('Integration | Component | rose/nav/primary', function(hooks) {
+module('Integration | Component | rose/nav/sidebar', function(hooks) {
   setupRenderingTest(hooks);
 
   test('it renders', async function(assert) {
     assert.expect(2)
     await render(hbs`
-      <Rose::Nav::Primary @title="Title" as |nav|>
+      <Rose::Nav::Sidebar @title="Title" as |nav|>
         <nav.link @route="index">Item Name</nav.link>
-      </Rose::Nav::Primary>
+      </Rose::Nav::Sidebar>
     `);
     assert.equal(find('.rose-nav-title').textContent.trim(), 'Title');
     assert.equal(find('.rose-nav-link').textContent.trim(), 'Item Name');
@@ -20,15 +20,14 @@ module('Integration | Component | rose/nav/primary', function(hooks) {
   test('it associates title with nav via aria-labelledby', async function (assert) {
     assert.expect(2)
     await render(hbs`
-      <Rose::Nav::Primary @title="Title" as |nav|>
+      <Rose::Nav::Sidebar @title="Title" as |nav|>
         <nav.link @route="index">Item Name</nav.link>
-      </Rose::Nav::Primary>
+      </Rose::Nav::Sidebar>
     `);
-    const el = find('.rose-nav-primary');
+    const el = find('.rose-nav-sidebar');
     const id = el.id;
     const titleId = `title-${id}`;
-    const titleEl = find('.rose-nav-title');
-    assert.equal(titleEl.id, titleId);
+    assert.equal(find('.rose-nav-title').id, titleId);
     assert.equal(el.getAttribute('aria-labelledby').trim(), titleId);
   });
 });
diff --git a/ui/core/app/styles/app.scss b/ui/core/app/styles/app.scss
index 64fed4405..667cc9039 100644
--- a/ui/core/app/styles/app.scss
+++ b/ui/core/app/styles/app.scss
@@ -3,7 +3,7 @@
 @import 'rose';
 @import 'notify';
 
-.rose-nav-primary {
+.rose-nav-sidebar {
   margin-top: sizing.rems(xxl);
 }
 
diff --git a/ui/core/app/templates/application.hbs b/ui/core/app/templates/application.hbs
index d9bc7b05e..5d66d58f1 100644
--- a/ui/core/app/templates/application.hbs
+++ b/ui/core/app/templates/application.hbs
@@ -33,10 +33,10 @@
   </page.main>
 
   <page.sidebar>
-    <Rose::Nav::Primary @title={{t 'dev.title'}} as |nav|>
+    <Rose::Nav::Sidebar @title={{t 'dev.title'}} as |nav|>
       <nav.link @route="orgs" class="active">{{t 'dev.item-name'}}</nav.link>
       <nav.link @route="index">{{t 'dev.item-name'}}</nav.link>
-    </Rose::Nav::Primary>
+    </Rose::Nav::Sidebar>
   </page.sidebar>
 
 </Rose::Layout::Page>