fix: simplify breadcrumb and change text

severo · Apr 14, 2019 · aa89980 · aa89980
1 parent a0c89cd
commit aa89980
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 34 deletions.
diff --git a/docs/css/main.css b/docs/css/main.css
diff --git a/docs/index.html b/docs/index.html
@@ -7,8 +7,8 @@
     <title>Quem bebe agrotóxicos?</title>
     <!-- TODO: bundle at build -->
     <script defer src="https://bundle.run/fuzzball@1.2.0"></script>
-    <script defer src="lib/main.pt.js?v=1555255170"></script>
-    <link rel="stylesheet" href="css/main.css?v=1555255170" />
+    <script defer src="lib/main.pt.js?v=1555256418"></script>
+    <link rel="stylesheet" href="css/main.css?v=1555256418" />
   </head>
   <body>
     <section id="page-title" class="hero is-small is-primary cocktail">
@@ -87,11 +87,7 @@ <h2 class="subtitle is-4">
                 ></button>
               </div>
             </section>
-            <nav
-              id="breadcrumb"
-              class="breadcrumb"
-              aria-label="breadcrumbs"
-            ></nav>
+            <nav id="breadcrumb"></nav>
             <section class="is-loading" id="details">
               <section class="content" id="details-main"></section>
               <footer id="details-footer">

diff --git a/docs/lib/main.pt.js b/docs/lib/main.pt.js
diff --git a/src/_javascript/breadcrumb/index.js b/src/_javascript/breadcrumb/index.js
@@ -21,34 +21,17 @@ export function makeBreadcrumb(parent, dispatcher, state) {
 
 function makeBrazil(parent) {
   parent.html(null);
-  parent
-    .append('ul')
-    .append('li')
-    .classed('is-active', true)
-    .classed('is-hidden', true)
-    .append('a')
-    .attr('href', '#')
-    .attr('aria-current', 'page')
-    .text('{{breadcrumb.brazil}}');
 }
 
 function makeMun(parent, dispatcher, data, mun) {
   parent.html(null);
-  const ul = parent.append('ul');
-  ul.append('li')
+  parent
     .append('a')
     .attr('href', '#')
-    .text('{{breadcrumb.brazil}}')
+    .text('{{breadcrumb.goto-brazil}}')
     .on('click', (ft, element) => {
-      // invoke callbacks
       dispatcher.call('breadcrumb-click-brazil', null, data);
     });
-  ul.append('li')
-    .classed('is-active', true)
-    .append('a')
-    .attr('href', '#')
-    .attr('aria-current', 'page')
-    .text(mun.properties.name);
 }
 
 function startLoading(element) {

diff --git a/src/_sass/main.scss b/src/_sass/main.scss
@@ -158,6 +158,10 @@ html {
   }
 }
 
+#breadcrumb {
+  margin-bottom: 1rem;
+}
+
 #details {
   #details-main {
     min-height: 150px;

diff --git a/src/index.mustache.html b/src/index.mustache.html
@@ -87,11 +87,7 @@ <h2 class="subtitle is-4">
                 ></button>
               </div>
             </section>
-            <nav
-              id="breadcrumb"
-              class="breadcrumb"
-              aria-label="breadcrumbs"
-            ></nav>
+            <nav id="breadcrumb"></nav>
             <section class="is-loading" id="details">
               <section class="content" id="details-main"></section>
               <footer id="details-footer">

diff --git a/src/lang/pt.json b/src/lang/pt.json
@@ -1,6 +1,6 @@
 {
   "breadcrumb": {
-    "brazil": "Brasil"
+    "goto-brazil": "Voltar para o mapa do Brasil"
   },
   "details": {
     "cocktail": {