Skip to content

Commit

Permalink
Merge branch 'develop'
Browse files Browse the repository at this point in the history
  • Loading branch information
jakubito committed Aug 10, 2021
2 parents 5ffff62 + 7e24ffe commit 6cbe144
Show file tree
Hide file tree
Showing 9 changed files with 35 additions and 27 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "spotify-release-list",
"author": "Jakub Dobes <[email protected]>",
"version": "2.0.1",
"version": "2.0.2",
"private": true,
"repository": "github:jakubito/spotify-release-list",
"license": "ISC",
Expand Down
2 changes: 1 addition & 1 deletion src/components/filters/AlbumGroupsFilter.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ function AlbumGroupsFilter() {
}

return (
<div className="AlbumGroupsFilter filter">
<div className="AlbumGroupsFilter Filters__filter">
{AlbumGroupLabels.filter(([group]) => releasesGroupMap[group]).map(([group, label]) => (
<Button
title={label}
Expand Down
2 changes: 1 addition & 1 deletion src/components/filters/DateRangeFilter.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ function DateRangeFilter() {
}, [values])

return (
<div className={classNames('DateRangeFilter', 'filter', { focused: focus })}>
<div className={classNames('DateRangeFilter Filters__filter', { focused: focus })}>
<Media query={{ maxWidth: 425 }}>
{(matches) => (
<DateRangePicker
Expand Down
18 changes: 11 additions & 7 deletions src/components/filters/DuplicatesFilter.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,17 @@ function DuplicatesFilter() {
const exclude = useSelector(getFiltersExcludeDuplicates)

return (
<Checkbox
id="duplicatesFilter"
label="Exclude duplicates"
defaultChecked={exclude}
onChange={(event) => defer(dispatch, setFilters({ excludeDuplicates: event.target.checked }))}
dark
/>
<div className="Filters__filter Filters__filter--inline">
<Checkbox
id="duplicatesFilter"
label="Exclude duplicates"
defaultChecked={exclude}
onChange={(event) =>
defer(dispatch, setFilters({ excludeDuplicates: event.target.checked }))
}
dark
/>
</div>
)
}

Expand Down
2 changes: 1 addition & 1 deletion src/components/filters/SearchFilter.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ function SearchFilter() {
)

return (
<div className="SearchFilter filter">
<div className="SearchFilter Filters__filter">
<Input
value={value}
onChange={onChange}
Expand Down
20 changes: 11 additions & 9 deletions src/components/filters/VariousArtistsFilter.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,17 @@ function VariousArtistsFilter() {
const exclude = useSelector(getFiltersExcludeVariousArtists)

return (
<Checkbox
id="variousArtistsFilter"
label="Exclude Various Artists"
defaultChecked={exclude}
onChange={(event) =>
defer(dispatch, setFilters({ excludeVariousArtists: event.target.checked }))
}
dark
/>
<div className="Filters__filter Filters__filter--inline">
<Checkbox
id="variousArtistsFilter"
label="Exclude Various Artists"
defaultChecked={exclude}
onChange={(event) =>
defer(dispatch, setFilters({ excludeVariousArtists: event.target.checked }))
}
dark
/>
</div>
)
}

Expand Down
6 changes: 3 additions & 3 deletions src/state/selectors.js
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,6 @@ export const getLastSyncDate = createSelector(
(lastSync, lastAutoSync) => {
if (lastSync || lastAutoSync) {
const newer = (lastSync || '') > (lastAutoSync || '') ? lastSync : lastAutoSync

return new Date(newer)
}

Expand Down Expand Up @@ -256,8 +255,9 @@ const getNoDuplicatesAlbumIds = createSelector(getOriginalReleases, (releases) =
const namesMap = {}

for (const album of albums) {
if (album.name in namesMap) continue
namesMap[album.name] = album.id
const name = album.name.toLowerCase()
if (name in namesMap) continue
namesMap[name] = album.id
}

return ids.concat(Object.values(namesMap))
Expand Down
8 changes: 4 additions & 4 deletions src/styles/Filters.scss
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
.Filters {
padding: 15px 15px 15px 260px;
padding: 15px 15px 0 260px;
margin-bottom: 5px;
background-color: $black-bis;

@include until-widescreen {
padding-left: 15px;
}

.filter {
&__filter {
display: flex;
margin-bottom: 15px;
align-items: center;

&:last-child {
margin-bottom: 0;
&--inline {
display: inline-block;
}
}
}
2 changes: 2 additions & 0 deletions src/styles/UpdateMessage.scss
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
font-size: 16px;
padding: 0 8px;
text-decoration: underline !important;
white-space: normal;
height: auto;

&:focus {
color: $white-bis !important;
Expand Down

0 comments on commit 6cbe144

Please sign in to comment.