History

Small improvements to notifications

Thibaut Courouble committed 10 years ago

Commit 4c2a5561 parent 7aab65b7 files · +13 −5Browse files

Unified · Split

7 files
  1. assets/javascripts/templates/notif_tmpl.coffee +1 −1
  2. assets/javascripts/views/misc/news.coffee +1 −1
  3. assets/javascripts/views/misc/notif.coffee +1 −0
  4. assets/javascripts/views/misc/updates.coffee +3 −0
  5. assets/stylesheets/components/_notif.scss +5 −1
  6. assets/stylesheets/global/_variables.scss +1 −1
  7. assets/stylesheets/global/variables-dark.scss +1 −1
assets/javascripts/templates/notif_tmpl.coffee+1−1
@@ -1,5 +1,5 @@
1notif = (title, html) ->1notif = (title, html) ->
2 html = html.replace /<a/g, '<a class="_notif-link"'2 html = html.replace /<a /g, '<a class="_notif-link" '
3 """<h5 class="_notif-title">#{title}</h5>#{html}<a href="#" class="_notif-close"></a>"""3 """<h5 class="_notif-title">#{title}</h5>#{html}<a href="#" class="_notif-close"></a>"""
4 4
5textNotif = (title, message) ->5textNotif = (title, message) ->
assets/javascripts/views/misc/news.coffee+1−1
@@ -4,7 +4,7 @@ class app.views.News extends app.views.Notif · 3 unmodified lines
4 @className += ' _notif-news'4 @className += ' _notif-news'
5 5
6 @defautOptions:6 @defautOptions:
7 autoHide: null7 autoHide: 30000
8 8
9 init: ->9 init: ->
10 @unreadNews = @getUnreadNews()10 @unreadNews = @getUnreadNews()
assets/javascripts/views/misc/notif.coffee+1−0
@@ -48,6 +48,7 @@ class app.views.Notif extends app.View · 47 unmodified lines
48 return48 return
49 49
50 onClick: (event) =>50 onClick: (event) =>
51 return if event.which isnt 1
51 if event.target.tagName isnt 'A' or event.target.classList.contains('_notif-close')52 if event.target.tagName isnt 'A' or event.target.classList.contains('_notif-close')
52 $.stopEvent(event)53 $.stopEvent(event)
53 @hide()54 @hide()
assets/javascripts/views/misc/updates.coffee+3−0
@@ -3,6 +3,9 @@ · 2 unmodified lines
3class app.views.Updates extends app.views.Notif3class app.views.Updates extends app.views.Notif
4 @className += ' _notif-news'4 @className += ' _notif-news'
5 5
6 @defautOptions:
7 autoHide: 30000
8
6 init: ->9 init: ->
7 @updatedDocs = @getUpdatedDocs()10 @updatedDocs = @getUpdatedDocs()
8 @show() if @updatedDocs.length11 @show() if @updatedDocs.length
assets/stylesheets/components/_notif.scss+5−1
@@ -29,13 +29,14 @@ · 28 unmodified lines
29 29
30._notif-info {30._notif-info {
31 float: right;31 float: right;
32 color: $textColorLight;32 color: $notifColorLight;
33}33}
34 34
35._notif-link,35._notif-link,
36._notif-link:hover {36._notif-link:hover {
37 color: inherit;37 color: inherit;
38 text-decoration: underline;38 text-decoration: underline;
39 cursor: pointer;
39}40}
40 41
41._notif-close {42._notif-close {
@@ -114,6 +115,7 @@ · 72 unmodified lines
114}115}
115 116
116._notif-list {117._notif-list {
118 margin: 0;
117 padding-left: 1rem;119 padding-left: 1rem;
118}120}
119 121
@@ -121,4 +123,6 @@ · 1 unmodified line
121 color: $textColor;123 color: $textColor;
122 background: rgba($tipBackground, .95);124 background: rgba($tipBackground, .95);
123 border: 1px solid $tipBorder;125 border: 1px solid $tipBorder;
126
127 ._notif-info { color: $textColorLight; }
124}128}
assets/stylesheets/global/_variables.scss+1−1
@@ -72,7 +72,7 @@ $labelBackground: #f4f4f4; · 71 unmodified lines
72 72
73$notifBackground: rgba(#333, .85);73$notifBackground: rgba(#333, .85);
74$notifColor: #fff;74$notifColor: #fff;
75$notifColorLight: #bbb;75$notifColorLight: #ccc;
76 76
77$tipBackground: #fffdcd;77$tipBackground: #fffdcd;
78$tipBorder: #e7dca9;78$tipBorder: #e7dca9;
assets/stylesheets/global/variables-dark.scss+1−1
@@ -72,7 +72,7 @@ $labelBackground: #3c3c3e; · 71 unmodified lines
72 72
73$notifBackground: rgba(#000, .85);73$notifBackground: rgba(#000, .85);
74$notifColor: #fff;74$notifColor: #fff;
75$notifColorLight: #bbb;75$notifColorLight: #ccc;
76 76
77$tipBackground: #3d4449;77$tipBackground: #3d4449;
78$tipBorder: #566166;78$tipBorder: #566166;