consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix color changing on hover on non-link notifications

Browse Source 
IMHO it was really strange to change the color on hover when the element
isn't interactive.
This commit is contained in:
Javi Martín
2022-10-09 23:58:36 +02:00
parent 9dfff1864a
commit 21faad66c9
2 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
app/assets/stylesheets/layout.scss
View File

@@ -1375,16 +1375,20 @@ form {
} }
} }
.notification-link {
color: inherit;
&:hover {
color: $link;
}
}
&:hover { &:hover {
a { a {
text-decoration: none; text-decoration: none;
} }
p:not(.time) {
color: $link;
}
&::before { &::before {
content: "\43"; content: "\43";
} }
@@ -1405,7 +1409,6 @@ form {
} }
p { p {
color: $text;
margin-bottom: 0; margin-bottom: 0;
max-width: 80%; max-width: 80%;
} }

2
app/views/notifications/_notification.html.erb
View File

@@ -5,7 +5,7 @@
title: notification.notifiable_title, title: notification.notifiable_title,
body: notification.notifiable_body } %> body: notification.notifiable_body } %>
<% link_text = render "/notifications/notification_body", locals %> <% link_text = render "/notifications/notification_body", locals %>
<%= link_to_if notification.link.present?, link_text, notification.link %> <%= link_to_if notification.link.present?, link_text, notification.link, class: "notification-link" %>
<% else %> <% else %>
<p> <p>
<strong> <strong>