Skip to content

Commit

Permalink
Remove duplicate dl.leaders rules in site.scss (#1493)
Browse files Browse the repository at this point in the history
> The removed section is a duplicate of the following section, minus one
additional rule at the end:
>
> https://github.com/Weasyl/weasyl/blob/6fd9b5969b67188e75baa09eeee0255bd36373b2/assets/scss/site.scss#L639-L669

Reviewed-by: Charmander <~@charmander.me>

I looked back in the old repository, and this seems to have been the result of a merge where one branch moved the styles and added the `dl + h4, #uc-info` rule, with the commit message “Fix commission information headers” (2014-02-04). Hopefully that was just to have everything in one place and there wasn’t/isn’t some style that depends on order – everything looks right to me on the two pages that use `dl.leaders`, anyway.
  • Loading branch information
charmander authored Dec 17, 2024
2 parents e3122a4 + bc39054 commit 10feb9c
Showing 1 changed file with 0 additions and 26 deletions.
26 changes: 0 additions & 26 deletions assets/scss/site.scss
Original file line number Diff line number Diff line change
Expand Up @@ -42,32 +42,6 @@ a:hover {
display: none;
}

/* single, two-column dl list items with dashed borders between them */
dl.leaders dt + dd {
border-top: 1px dashed #b4b4b4;
margin-top: 0.375em;
padding-top: 0.375em;
}

dl.leaders dt:first-child + dd {
border-top: none;
}

dl.leaders dt {
clear: left;
float: left;
font-weight: 700;
padding-top: 0.375em;
}

dl.leaders dd + dt, dl.leaders dt + dt {
padding-top: 0.75em;
}

dl.leaders dd {
text-align: right;
}

/* Typefaces
--------------------------------*/
#page-container, textarea, .typeface-default, button {
Expand Down

0 comments on commit 10feb9c

Please sign in to comment.