Issue 6484 - Add horizontal-list class to website-defaults

static/scss/utilities/_lists.scss

+// This file is part of website-defaults
+// Copyright (C) 2016-present eyeo GmbH
+//
+// website-defaults is free software: you can redistribute it and/or
+// modify it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// website-defaults is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with website-defaults.  If not, see <http://www.gnu.org/licenses/>.
+
+/*******************************************************************************
+ * Lists
+ ******************************************************************************/
+
+/* Horizontal List
+ ******************************************************************************/
+
+.horizontal-list
+{
+  display: inline;
+  list-style: none;
+}
+
+.horizontal-list,
+.content .horizontal-list,

[juliandoucette, 2018/03/24 15:48:09]

I think it's OK (maybe even preferable?) to use !important for utility classes
instead of adding .content specific selectors.

[ire, 2018/03/26 06:59:39]

I don't think we should use !important unless it is something that *all*
.horizontal-list *must* have. I don't think zero padding is necessarily this, I
can easily imagine cases where we may want to change the padding of a horizontal
list. 

+[dir="rtl"] .horizontal-list
+{
+  padding: 0;
+}
+
+.horizontal-list li

[juliandoucette, 2018/03/24 15:48:09]

Why not just add this selector to .horizontal-list above? :)

[ire, 2018/03/26 06:59:39]

Haha don't know how I missed that! Thanks, Done.

+{
+  display: inline;
+  list-style: none;
+}