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.

Sign up for GitHub

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

Jump to bottom

When not logged in, responding to comments should redirect to /remote_interact #559

Open
marek-lach opened this issue May 1, 2019 · 1 comment
Open

When not logged in, responding to comments should redirect to /remote_interact #559

marek-lach opened this issue May 1, 2019 · 1 comment
Labels
A: Backend Code running on the server Design UI/UX related issues and PRs Rendering How elements're rendered out for the end user

Comments

@marek-lach
Copy link
Member

marek-lach commented May 1, 2019 •
edited
Loading

Is your feature request related to a problem? Please describe.
When not logged in, clicking on the Respond button on an article comment currently does nothing useful, only scrolls the user back to the beginning of the article. This does not make much sense...

Screenshot 2019-05-01 at 23 11 02

Describe the solution you'd like
Clicking on the 'Respond' button regarding a comment to a specific article should rather redirect to the .../remote_interact page regarding that article.

Screenshot 2019-05-01 at 23 18 16

Describe alternatives you've considered
**EDIT: not needed

Additional context
Not needed...
@marek-lach marek-lach added Design UI/UX related issues and PRs A: Backend Code running on the server Rendering How elements're rendered out for the end user P: Low labels May 1, 2019
@marek-lach marek-lach changed the title Responding to comments when not logged in should redirect to /remote_interact When not logged in, responding to comments should redirect to /remote_interact, have permaling May 8, 2019
@trinity-1686a
Copy link
Contributor

the "alternative" was implemented as part of redesign in #553 (but having a /remote_interact was not yet)

@marek-lach marek-lach changed the title When not logged in, responding to comments should redirect to /remote_interact, have permaling When not logged in, responding to comments should redirect to /remote_interact May 19, 2019
@elegaanz elegaanz removed the P: Low label Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: Backend Code running on the server Design UI/UX related issues and PRs Rendering How elements're rendered out for the end user
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@elegaanz @trinity-1686a @marek-lach