Skip to content
This repository has been archived by the owner on Jul 30, 2021. It is now read-only.

Commit

Permalink
fix: comments in the code
Browse files Browse the repository at this point in the history
  • Loading branch information
RitaOak committed May 27, 2020
1 parent b6e8144 commit 54e3394
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 7 deletions.
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
"prepublish": "npm run snyk-protect"
},
"dependencies": {
"@fortawesome/fontawesome-free": "^5.13.0",
"@material/button": "^3.0.0",
"@mdi/font": "^3.9.96",
"@mdi/svg": "^3.7.95",
Expand Down
9 changes: 8 additions & 1 deletion style/app.scss
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
// Code in the comments belongs to a previous iteration of the project that might need to be returned to


// Import a Google Font
// Open Sans - Regular (400)
// Open Sans - Bold (700)
Expand All @@ -6,7 +9,7 @@
$link: #0060df;

// Import Material Design
//@import "../node_modules/material-design-icons/iconfont/";
// @import "../node_modules/material-design-icons/iconfont/";

// Bulma *ALL SASS imports
@import 'bulma/sass/utilities/mixins';
Expand Down Expand Up @@ -132,6 +135,8 @@ body {
}

.column.is-narrow {
// Menu side bar white background and box shadow
// Code is commented in case we need to use this approach again
//background-color: white;
//box-shadow: 0 3px 6px rgba(0, 0, 0, 0.16), 0 3px 6px rgba(0, 0, 0, 0.23);
//z-index: 1;
Expand Down Expand Up @@ -164,6 +169,8 @@ body {

.menu {
width: 185px;
// Previous menu position, might be useful once we remove the top tabs bar from the default polymer project
// Code is commented in case we need to use this approach again
//position: fixed;
//top: 8.5em;
margin-top: 3.5em;
Expand Down
5 changes: 5 additions & 0 deletions style/topnav-buttons.scss
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
// Code in the comments belongs to a previous iteration of the project that might need to be returned to


.navbar-item {
background-color: transparent;
}
Expand All @@ -16,6 +19,8 @@
margin-right: 10px;
font-size: 14px;
height: 35px;
// Original top navbar styling
// Code is commented in case we need to use this approach again
//border-radius: 25px;
//border: solid;
//border-color: #EEEEEE;
Expand Down
5 changes: 0 additions & 5 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -646,11 +646,6 @@
lodash "^4.17.11"
to-fast-properties "^2.0.0"

"@fortawesome/fontawesome-free@^5.13.0":
version "5.13.0"
resolved "https://registry.yarnpkg.com/@fortawesome/fontawesome-free/-/fontawesome-free-5.13.0.tgz#fcb113d1aca4b471b709e8c9c168674fbd6e06d9"
integrity sha512-xKOeQEl5O47GPZYIMToj6uuA2syyFlq9EMSl2ui0uytjY9xbe8XS0pexNWmxrdcCyNGyDmLyYw5FtKsalBUeOg==

"@material/animation@^3.0.0":
version "3.0.0"
resolved "https://registry.yarnpkg.com/@material/animation/-/animation-3.0.0.tgz#bbea365f961204a499dcaf04c5c60548af45d03f"
Expand Down

1 comment on commit 54e3394

@datreeio
Copy link

@datreeio datreeio bot commented on 54e3394 May 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6 / 1 Rules Failed

:octocat: Ensure a .gitignore file is included in project

Please sign in to comment.