Skip to content

Commit

Permalink
Fixed layout and localization
Browse files Browse the repository at this point in the history
Added a button to test the notification at the customizer.

Also fixed typo for locale: `background_notificationPlayPotplayer` and
removed grunt generated files.
  • Loading branch information
Wassup789 committed Sep 24, 2016
1 parent fdf891c commit 0fa3927
Show file tree
Hide file tree
Showing 8 changed files with 78 additions and 26,807 deletions.
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
# Remove auto-generated files
settings.html
js/settings.js

pages/edit-notification.js
pages/edit-notification.html

# Windows image file caches
Thumbs.db
Expand Down
9 changes: 9 additions & 0 deletions .idea/codeStyleSettings.xml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

18 changes: 15 additions & 3 deletions _locales/en/messages.json
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@
},
"background_notificationPlayPotplayer": {
"description": "",
"message": "Play with Potplayer"
"message": "Play with PotPlayer"
},
"background_notificationLogCheck": {
"description": "",
Expand Down Expand Up @@ -408,7 +408,19 @@
"message": "Remove from YouTube Notifications"
},
"notificationCustomizer_buttons_title": {
"description": "The title for the notification customizer tools",
"description": "The title for the notification customizer button card",
"message": "Available Buttons"
}
},
"notificationCustomizer_headerTitle": {
"description": "The title for the notification customizer header",
"message": "Drag and drop buttons from the list to the notification on the right."
},
"notificationCustomizer_headerSubtitle": {
"description": "The subtitle for the notification customizer header",
"message": "Notification buttons are limited to two buttons."
},
"notificationCustomizer_buttonSave": {
"description": "The text for the paper-button save in the notification customizer page",
"message": "Save"
}
}
29 changes: 22 additions & 7 deletions pages/edit-notification.css
Original file line number Diff line number Diff line change
Expand Up @@ -190,9 +190,6 @@ a, .link{
margin-left: 16px;
}

#notification {
transition: transform 0.1s;
}
#content {
display: flex;
flex-direction: column;
Expand All @@ -205,18 +202,26 @@ a, .link{
flex-direction: row;
justify-content: center;
align-items: center;
height: 100vh;
height: calc(100vh - 64px);
}
#content_container > * {
display: flex;
width: 50vw;
justify-content: center;
align-items: center;
}

#notification_container {
display: flex;
justify-content: center;
}

#notificationCustomizer_card_container {
flex-direction: column;
}
#notificationCustomizer_card_container paper-card {
margin: 10px 0;
}
#notificationCustomizer_buttons {
width: 80%;
}
Expand Down Expand Up @@ -270,17 +275,17 @@ a, .link{
padding: 20px 40px;
}

#header * {
#header > * {
display: block;
text-align: center;
}

#header span:nth-child(1) {
#header .title {
font-size: 28px;
font-weight: 300;
}

#header span:nth-child(2) {
#header .subtitle {
color: #797979;
font-size: 16px;
font-weight: 300;
Expand All @@ -294,3 +299,13 @@ a, .link{
border-radius: 5px;
margin-top: 20px;
}

.notification_separator {
padding: 15px 0 5px 0;
font-size: 21px;
border-bottom: 1px solid #eaeaea;
}

.notification_separator:nth-child(1) {
padding-top: 0;
}
7,393 changes: 0 additions & 7,393 deletions pages/edit-notification.html

This file was deleted.

Loading

0 comments on commit 0fa3927

Please sign in to comment.