From 323f187ccfe2aa46183d69aaf50f57c567da6019 Mon Sep 17 00:00:00 2001
From: hanakin <midaym@gmail.com>
Date: Tue, 25 Apr 2017 18:07:21 -0400
Subject: [PATCH] [ticket/15192] Cleanup selectors and proper scope

PHPBB3-15192
---
 .../template/notification_dropdown.html       | 12 ++---
 phpBB/styles/prosilver/theme/bidi.css         | 10 ++--
 phpBB/styles/prosilver/theme/colours.css      | 10 ++--
 phpBB/styles/prosilver/theme/common.css       | 46 +++++++++----------
 4 files changed, 39 insertions(+), 39 deletions(-)

diff --git a/phpBB/styles/prosilver/template/notification_dropdown.html b/phpBB/styles/prosilver/template/notification_dropdown.html
index 177160b1b8..31fbb1d4cf 100644
--- a/phpBB/styles/prosilver/template/notification_dropdown.html
+++ b/phpBB/styles/prosilver/template/notification_dropdown.html
@@ -3,7 +3,7 @@
 	<div class="dropdown-contents">
 		<div class="header">
 			{L_NOTIFICATIONS}
-			<span class="header_settings">
+			<span class="header-settings">
 				<a href="{U_NOTIFICATION_SETTINGS}">{L_SETTINGS}</a>
 				<!-- IF NOTIFICATIONS_COUNT -->
 					<span id="mark_all_notifications"> &bull; <a href="{U_MARK_ALL_NOTIFICATIONS}" data-ajax="notification.mark_all_read">{L_MARK_ALL_READ}</a></span>
@@ -11,19 +11,19 @@
 			</span>
 		</div>
 
-		<ul>
+		<ul class="dropdown-extended-list notification-list">
 			<!-- IF not .notifications -->
-				<li class="no_notifications">
+				<li class="dropdown-extended-item no-notifications">
 					{L_NO_NOTIFICATIONS}
 				</li>
 			<!-- ENDIF -->
 			<!-- BEGIN notifications -->
-				<li class="<!-- IF notifications.UNREAD --> bg2<!-- ENDIF --><!-- IF notifications.STYLING --> {notifications.STYLING}<!-- ENDIF --><!-- IF not notifications.URL --> no-url<!-- ENDIF -->">
+				<li class="dropdown-extended-item notification-item<!-- IF notifications.UNREAD --> bg2<!-- ENDIF --><!-- IF notifications.STYLING --> {notifications.STYLING}<!-- ENDIF --><!-- IF not notifications.URL --> no-url<!-- ENDIF -->" <!-- IF notifications.UNREAD -->data-notification-unread="true"<!-- ENDIF -->>
 					<!-- IF notifications.URL -->
 						<a class="notification-block" href="<!-- IF notifications.UNREAD -->{notifications.U_MARK_READ}" data-real-url="{notifications.URL}<!-- ELSE -->{notifications.URL}<!-- ENDIF -->">
 					<!-- ENDIF -->
-						<!-- IF notifications.AVATAR -->{notifications.AVATAR}<!-- ELSE --><img src="{T_THEME_PATH}/images/no_avatar.gif" alt="" /><!-- ENDIF -->
-						<div class="notification_text">
+						<!-- IF notifications.AVATAR -->{notifications.AVATAR}<!-- ELSE --><img class="notification-avatar" src="{T_THEME_PATH}/images/no_avatar.gif" alt="" /><!-- ENDIF -->
+						<div class="notification-text">
 							<p class="notification-title">{notifications.FORMATTED_TITLE}</p>
 							<!-- IF notifications.REFERENCE --><p class="notification-reference">{notifications.REFERENCE}</p><!-- ENDIF -->
 							<!-- IF notifications.FORUM --><p class="notification-forum">{notifications.FORUM}</p><!-- ENDIF -->
diff --git a/phpBB/styles/prosilver/theme/bidi.css b/phpBB/styles/prosilver/theme/bidi.css
index a5b9b51dbf..4b0c9f4419 100644
--- a/phpBB/styles/prosilver/theme/bidi.css
+++ b/phpBB/styles/prosilver/theme/bidi.css
@@ -126,7 +126,7 @@
 	text-align: right;
 }
 
-.rtl .dropdown-extended .header .header_settings,
+.rtl .dropdown-extended .header .header-settings,
 .rtl .dropdown-container-right {
 	float: left;
 }
@@ -138,23 +138,23 @@
 
 /* Notifications
 ----------------------------------------- */
-.rtl .notification_list ul li img {
+.rtl .notification-avatar {
 	float: right;
 	margin-right: 0;
 	margin-left: 5px;
 }
 
-.rtl .notification_list div.notifications {
+.rtl .notification-menu .notifications {
 	margin-right: 50px;
 	margin-left: 0;
 }
 
-.rtl .notification_text {
+.rtl .notification-text {
 	margin-right: 58px;
 	margin-left: 0;
 }
 
-.rtl .notification_list p.notification-time {
+.rtl .notification-time {
 	text-align: left;
 }
 
diff --git a/phpBB/styles/prosilver/theme/colours.css b/phpBB/styles/prosilver/theme/colours.css
index 87b1030a06..587136e13f 100644
--- a/phpBB/styles/prosilver/theme/colours.css
+++ b/phpBB/styles/prosilver/theme/colours.css
@@ -1102,11 +1102,11 @@ input.disabled {
 	}
 }
 
-.dropdown-extended ul li {
+.dropdown-extended .dropdown-extended-item {
 	border-top-color: #b9b9b9;
 }
 
-.dropdown-extended ul li:hover {
+.dropdown-extended .dropdown-extended-item:hover {
 	background-color: #cfe1f6;
 	color: #000000;
 }
@@ -1164,12 +1164,12 @@ input.disabled {
 
 /* notifications */
 
-.notification_list p.notification-time {
+.notification-time {
 	color: #4c5d77;
 }
 
-li.notification-reported strong,
-li.notification-disapproved strong {
+.notification-reported strong,
+.notification-disapproved strong {
 	color: #d31141;
 }
 
diff --git a/phpBB/styles/prosilver/theme/common.css b/phpBB/styles/prosilver/theme/common.css
index b5c2cfc5f6..d1335314ab 100644
--- a/phpBB/styles/prosilver/theme/common.css
+++ b/phpBB/styles/prosilver/theme/common.css
@@ -1160,7 +1160,7 @@ form > p.post-notice strong {
 .inner:after,
 ul.linklist:after,
 .action-bar:after,
-.notification_text:after,
+.notification-text:after,
 .tabs-container:after,
 .tabs > ul:after,
 .minitabs > ul:after,
@@ -1200,14 +1200,14 @@ ul.linklist:after,
 	display: none;
 }
 
-.dropdown-extended ul {
+.dropdown-extended .dropdown-extended-list {
 	clear: both;
 	overflow-x: hidden;
 	overflow-y: auto;
 	max-height: 350px;
 }
 
-.dropdown-extended ul li {
+.dropdown-extended .dropdown-extended-item {
 	font-size: 0.95em;
 	border-top: 1px solid;
 	position: relative;
@@ -1218,11 +1218,11 @@ ul.linklist:after,
 	list-style-type: none;
 }
 
-.dropdown-extended ul li:first-child {
+.dropdown-extended .dropdown-extended-item:first-child {
 	border-top: none;
 }
 
-.dropdown-extended ul li.no_notifications {
+.dropdown-extended .dropdown-extended-item.no-notifications {
 	padding: 10px;
 }
 
@@ -1249,13 +1249,13 @@ ul.linklist:after,
 	padding: 0 10px;
 }
 
-.dropdown-extended .header .header_settings {
+.dropdown-extended .header .header-settings {
 	font-weight: normal;
 	text-transform: none;
 	float: right;
 }
 
-.dropdown-extended .header .header_settings a {
+.dropdown-extended .header .header-settings a {
 	display: inline-block;
 	padding: 0 5px;
 }
@@ -1271,8 +1271,8 @@ ul.linklist:after,
 	text-align: center;
 }
 
-.dropdown-extended ul li a,
-.dropdown-extended ul li.no-url {
+.dropdown-extended .notification-block,
+.dropdown-extended .dropdown-extended-item.no-url {
 	padding: 8px;
 }
 
@@ -1280,14 +1280,14 @@ ul.linklist:after,
 	padding: 5px 0;
 }
 
-.dropdown-extended ul li a,
-.notification_list dt > a,
+.dropdown-extended .dropdown-extended-item a,
+.notification-menu dt > a,
 .dropdown-extended .footer > a {
 	text-decoration: none;
 	display: block;
 }
 
-.notification_list ul li img {
+.notification-avatar {
 	float: left;
 	width: auto !important;
 	max-width: 50px;
@@ -1296,48 +1296,48 @@ ul.linklist:after,
 	margin-right: 5px;
 }
 
-.notification_list ul li p {
+.notification-item p {
 	font-size: 1em;
 	margin-bottom: 4px;
 }
 
-.notification_list p.notification-reference,
-.notification_list p.notification-location,
-.notification_list li a p.notification-reason {
+.notification-reference,
+.notification-location,
+.notification-reason {
 	white-space: nowrap;
 	text-overflow: ellipsis;
 	overflow: hidden;
 }
 
-.notification_list p.notification-time {
+.notification-time {
 	font-size: 0.9em;
 	text-align: right;
 	margin: 0;
 }
 
-.notification_list div.notifications {
+.notification-menu .notifications {
 	margin-left: 50px;
 	padding: 5px;
 }
 
-.notification_list div.notifications a {
+.notification-menu .notifications a {
 	display: block;
 }
 
-.notification_list p.notifications_title {
+.notifications-title {
 	font-family: "Trebuchet MS", Arial, Helvetica, sans-serif;
 	font-size: 1.2em !important;
 }
 
-.notification_list p.notifications_title strong {
+.notifications-title strong {
 	font-weight: bold;
 }
 
-.notification_list p.notifications_time {
+.notifications-time {
 	font-size: 0.9em !important;
 }
 
-.notification_text {
+.notification-text {
 	margin-left: 58px;
 }