Skip to content

Feature: Match Library and playlists tableviews with design - #132

Merged
GianniCarlo merged 3 commits into
TortugaPower:next/majorfrom
pichfl:feature/align-library
Jun 4, 2018
Merged

GianniCarlo merged 3 commits into
TortugaPower:next/majorfrom
pichfl:feature/align-library

Conversation

@pichfl

@pichfl pichfl commented Jun 3, 2018

Copy link
Copy Markdown
Collaborator

Re #129

@pichfl pichfl added this to the v3.0.0 milestone Jun 3, 2018
@pichfl
pichfl requested a review from GianniCarlo June 3, 2018 20:25
@GianniCarlo

Copy link
Copy Markdown
Collaborator

There's a problem with the play/pause icon in the mini player
img_1021

@pichfl

pichfl commented Jun 4, 2018

Copy link
Copy Markdown
Collaborator Author

Seems I failed to fix the constraints, on it.

Florian Pichler added 3 commits June 4, 2018 11:04
- Adjusted spacing and font sizes
- centered progress
- prepared BookCell for future enhancements
@GianniCarlo
GianniCarlo merged commit 1fefce7 into TortugaPower:next/major Jun 4, 2018
@pichfl
pichfl deleted the feature/align-library branch June 5, 2018 09:24
GianniCarlo added a commit that referenced this pull request Oct 5, 2026
A progress push now carries the book's lastPlayDate, as the Android app's does
(bookplayer-android #130 and #132). The queued task already stored the date,
but the push never read it.

- Jellyfin gets LastPlayedDate. It stores a date only when a client sends one.
- AudiobookShelf gets lastUpdate, in epoch milliseconds. Without it, ABS
  records when the push landed. It honors ours when updating an existing
  entry; a book's first write still takes the server's clock.

The date is persisted with the task, like the position, so a stored value no
real date has (NaN, infinite, past the year 5000) is left out instead of
trapping the conversion on every pop. A task without a date sends none, and
the server keeps or stamps its own.
GianniCarlo added a commit that referenced this pull request Oct 6, 2026
A progress push now carries the book's lastPlayDate, as the Android app's does
(bookplayer-android #130 and #132). The queued task already stored the date,
but the push never read it.

- Jellyfin gets LastPlayedDate. It stores a date only when a client sends one.
- AudiobookShelf gets lastUpdate, in epoch milliseconds. Without it, ABS
  records when the push landed. It honors ours when updating an existing
  entry; a book's first write still takes the server's clock.

The date is persisted with the task, like the position, so a stored value no
real date has (NaN, infinite, past the year 5000) is left out instead of
trapping the conversion on every pop. A task without a date sends none, and
the server keeps or stamps its own.
GianniCarlo added a commit that referenced this pull request Oct 7, 2026
A progress push now carries the book's lastPlayDate, as the Android app's does
(bookplayer-android #130 and #132). The queued task already stored the date,
but the push never read it.

- Jellyfin gets LastPlayedDate. It stores a date only when a client sends one.
- AudiobookShelf gets lastUpdate, in epoch milliseconds. Without it, ABS
  records when the push landed. It honors ours when updating an existing
  entry; a book's first write still takes the server's clock.

The date is persisted with the task, like the position, so a stored value no
real date has (NaN, infinite, past the year 5000) is left out instead of
trapping the conversion on every pop. A task without a date sends none, and
the server keeps or stamps its own.
GianniCarlo added a commit that referenced this pull request Oct 10, 2026
A progress push now carries the book's lastPlayDate, as the Android app's does
(bookplayer-android #130 and #132). The queued task already stored the date,
but the push never read it.

- Jellyfin gets LastPlayedDate. It stores a date only when a client sends one.
- AudiobookShelf gets lastUpdate, in epoch milliseconds. Without it, ABS
  records when the push landed. It honors ours when updating an existing
  entry; a book's first write still takes the server's clock.

The date is persisted with the task, like the position, so a stored value no
real date has (NaN, infinite, past the year 5000) is left out instead of
trapping the conversion on every pop. A task without a date sends none, and
the server keeps or stamps its own.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants