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

Scrollbar bug & more #429

Closed
Slluxx opened this issue Jul 5, 2023 · 9 comments
Closed

Scrollbar bug & more #429

Slluxx opened this issue Jul 5, 2023 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@Slluxx
Copy link

Slluxx commented Jul 5, 2023

Scrollbar bug

Create a new post and quickly move the mouse over the available icons in the wysiwyg editor. If you do it fast enough, multiple popups are open at the same time and one is forced into the next line, which not just looks weird, it can also shift the entire page because of a scrollbar appearing and then disappearing again. I tested this on a 1920x1080 monitor in fullscreen. If the scrollbar exists ahead of time, no noticible shift happens and the miss-placed popup is way harder to notice.

Visists counter

The counter goes up even if the page is just refreshed. This is not acceptable as it can be used to artificially blow posts up and annoy staff / visitors. There needs to be some kind of check to prevent this. This is not easy as guests should also count but there needs some kind of check in place. Maybe not IP but a few semi-unique combined pieces of information for guests and account ids for logged in users. Doesn't need a 100% waterproof solution but at least it needs to be thought about.

Multiple different OAuth provider

Probably belongs into answerdev/plugins but i'll note it here anyway.
Having one OAuth provider is not enough. I should be able to use as many as i want to and not be locked to one to increase the potential userbase.

Answering my own question inside the content, not a new answer.

If i have a simple question that i want to write an answer for, i am forced to put something inside the the "content" textbox. Instead of forcing me to write something super obvious or just repeating the question from the title, i should be allowed to put the answer in directly. One real example:

Title: "What are the reasons for a ban?"
Content: "Is there a list of things that i can avoid to get banned?"
Self-Answer: "Yes, [...]"

The person was forced to basically write the same question inside the content because the title was so self-explanatory that there was nothing of value to add. It doesn't look good, adds nothing of value and could be much more compact and readable if the person was just allowed to put the answer into that content block.

I get that not every question is that obvious and sometimes more context is needed but in hyper specialized communities, self-answered questions usually are that simple and should be accounted for.

@Slluxx Slluxx added the bug Something isn't working label Jul 5, 2023
@LinkinStars
Copy link
Member

@Slluxx About Multiple different OAuth provider. If you need more than one OAuth login platform, you can package several plugins. For example:

./answer build --with github.com/answerdev/plugins/connector/github  --with github.com/answerdev/plugins/connector/google

Detailed documentation can be found at https://answer.dev/docs/development/extending/

Thank you very much for your feedback, and recommended to divide this issue into multiple ones so that we can easily assign them to different members for processing.

@LinkinStars
Copy link
Member

LinkinStars commented Jul 5, 2023

@Slluxx
Copy link
Author

Slluxx commented Jul 5, 2023

@LinkinStars I thought about doing it like that but what if i want/need 2 OAuth Basic provider (because way more than just google and github exist)? If it actually is possible to build with multiple --with github.com/answerdev/plugins/connector/basic (which i doubt but did not test), it would become quite annoying in the admin pannel to distinguish between them. OAuth simple should have a way to add however many providers a user desires.

BTW i am using docker, so a way of adding plugins through the Webinterface would be a welcome feature.

Thanks for listening and accepting my feedback!

@Slluxx
Copy link
Author

Slluxx commented Jul 5, 2023

A few more things i noticed:

User deletion

Deleting a user does not really delete a user. Some information is still available in the dashboard and thus in the database. I should be able to prune/force delete to remove everything from that user. I am not sure what the best way to handle comments/answers on other users questions would be like (eg remove everything and replace with generic message or keep message and replace username with deleted-user, banned-user or whatever) but keeping certain userdata when specifically deleting a user is a no-go

Compliances (GDPR/EU/...)

Cookie notices, user information requests/dumps/deletions

Installations

When setting this up with a database and the database name can not be found, it should asked to create the database instead of throwing an error message.

Edit users

It should be possible to edit every users entire data (username, profile picture, bio, etc) instead of just editing their password.

@fenbox
Copy link
Member

fenbox commented Jul 24, 2023

Answering my own question inside the content, not a new answer.

Thank you for your suggestion. While we understand your point of view, Answer's design is intended to ensure high-quality content and readability. Therefore, we recommend separating questions and answers into different sections.

If your question and answer are very simple, you can provide the same content in both sections, which will not affect the quality or readability.

@Slluxx
Copy link
Author

Slluxx commented Jul 26, 2023

If your question and answer are very simple, you can provide the same content in both sections, which will not affect the quality or readability.

Answer's design is intended to ensure high-quality content and readability.

I strongly disagree. Text, that doesn't add any value is superfluous and therefore not "high-quality content".

Aside from this, id like to know if my previous comment was/is acknowledged in any way

@shuashuai
Copy link
Member

Create a new post and quickly move the mouse over the available icons in the wysiwyg editor. If you do it fast enough, multiple popups are open at the same time and one is forced into the next line,

Can you provide a video to reproduce this for fast mouse movement that opens multiple windows? You can go to our community to submit this video.

@Slluxx
Copy link
Author

Slluxx commented Aug 31, 2023

No, sorry. Since sqlite cant be reliably used and other issues and limitations that occurred, i moved away from answerdev and cant replicate this anymore without setting it back up. All you need to do is make the window a bit smaller and then move the mouse cursor over the icons in the WYSISYG editor. Hovering over an icon creates a popup, not a window, and you can move the mouse faster to another icon than the previous popup is closing, thus pushing one popup into the next line. I am not exactly sure how the window size plays into this but i assume that the scrollbar can appear because 2 popups make the site content larger in height and if content>window, the scrollbar gets enabled - making this whole thing just very ugly to look at.

It was easy to repeat and also very obvious, so just try it a few times.

Edit: Nvm i have an account on your community forum so i can just record it there.

Untitled.Project.mp4

@shuashuai
Copy link
Member

No, sorry. Since sqlite cant be reliably used and other issues and limitations that occurred, i moved away from answerdev and cant replicate this anymore without setting it back up. All you need to do is make the window a bit smaller and then move the mouse cursor over the icons in the WYSISYG editor. Hovering over an icon creates a popup, not a window, and you can move the mouse faster to another icon than the previous popup is closing, thus pushing one popup into the next line. I am not exactly sure how the window size plays into this but i assume that the scrollbar can appear because 2 popups make the site content larger in height and if content>window, the scrollbar gets enabled - making this whole thing just very ugly to look at.

It was easy to repeat and also very obvious, so just try it a few times.

Edit: Nvm i have an account on your community forum so i can just record it there.

Untitled.Project.mp4

Thanks for the feedback. This bug is caused by bootstrap tooltips. I will use the native title instead of tooltips. And fixed in v1.1.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants