Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mobile Next / Previous buttons #1406

Closed
baz4096 opened this issue Feb 2, 2023 · 6 comments
Closed

Mobile Next / Previous buttons #1406

baz4096 opened this issue Feb 2, 2023 · 6 comments
Milestone

Comments

@baz4096
Copy link

baz4096 commented Feb 2, 2023

On 2.18, when viewed on a narrow mobile device (iphone) there used to be a Next button that closed the current item, and opened the next item, without having to scroll back to the top of the article and saving an additional two keypresses - the equivalent of the 'J' keyboard shortcut on devices with keys... Where did this button go? On 2.19, we only have "Open" at the bottom of each article.
Also, I seem to recall a "Previous" button (I could be wrong!), which I used much less frequently, but still miss. Is it possible to restore these two buttons?

@jtojnar
Copy link
Member

jtojnar commented Feb 2, 2023

Looking at the style sheet, the button is hidden by default:

.entry-next {
display: none;
}

Then displayed for screens with width ≤ 1024px:

.entry-toolbar .entry-next {
display: block;
}

And then hidden again for width ≤ 641px:

.entry-toolbar .entry-next {
display: none;
}

I am not sure if this ever changed but you can always override this by adding the following to user.css file in your selfoss directory:

@media only screen and (max-width: 641px) {
    .entry-toolbar .entry-next {
        display: block;
    }
}

I do not recall any “Previous” button and was not able to find entry-prev in the source history.

@baz4096
Copy link
Author

baz4096 commented Feb 2, 2023

Thank you! I'm probably wrong about the previous button. Thanks for the tip. Next on the top bar isn't overly convenient on mobile devices - I much preferred it at the bottom where it's easy for your thumb to reach - however compared to no Next, at all, this is great. Many thanks.

@jtojnar
Copy link
Member

jtojnar commented Feb 2, 2023

Oh, right for showing it at the bottom, you can do:

@media only screen and (max-width: 641px) {
    .entry-smartphone-share .entry-next {
        display: block;
    }
}

@baz4096
Copy link
Author

baz4096 commented Feb 2, 2023

Now it's perfect again, many thanks 👍

@baz4096 baz4096 closed this as completed Feb 2, 2023
@jtojnar jtojnar reopened this Feb 2, 2023
@jtojnar jtojnar added this to the 2.20 milestone Feb 2, 2023
@jtojnar jtojnar closed this as completed in bc31b41 Feb 2, 2023
@jtojnar
Copy link
Member

jtojnar commented Feb 2, 2023

I found out why it regressed so I have decided to fix it in selfoss itself.

@thewholelifetolearn
Copy link

I have tried the latest snapshot. I have the button displayed, but when I go to the next feed, it doesn't mark as read the feed that has just been read.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants