Skip to content
New issue

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

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

Already on GitHub? # to your account

Unread is not visible in navigation sidebar #3642

Closed
Eldenroot opened this issue Sep 16, 2022 · 7 comments · Fixed by #3791
Closed

Unread is not visible in navigation sidebar #3642

Eldenroot opened this issue Sep 16, 2022 · 7 comments · Fixed by #3791
Assignees
Milestone

Comments

@Eldenroot
Copy link

Current Behavior

Unread text is not fully visible. Tested on Chrome and Edge

image

Steps to Reproduce

Go to some discussion with a lot of posts...

Expected Behavior

The unread text should be fully visible

Screenshots

image

Environment

  • Flarum version: 1.5.0 (flarum.org board)
  • Website URL: https://discuss.flarum.org/
  • Webserver: [e.g. apache, nginx]
  • Hosting environment: [e.g. shared, vps]
  • PHP version: x.y.z
  • Browser: [e.g. chrome 67, safari 11]

Output of php flarum info

Output of "php flarum info", run this in terminal in your Flarum directory.

Possible Solution

No response

Additional Context

No response

@luceos
Copy link
Member

luceos commented Sep 16, 2022

This has been the behavior since the earliest beta's. If we want to change this we need to Approve it first. Please use https://discuss.flarum.org/t/proposals to file it for consideration.

@luceos luceos closed this as completed Sep 16, 2022
@matteocontrini
Copy link
Contributor

Is a proposal and a discussion needed to decide whether it's ok to have cut text in the UI? 🤔 I don't see how this could not be considered a bug.

@davwheat
Copy link
Member

I also agree.

If we have a bug that has existed since early betas, that doesn't mean we should class it as expected behaviour now we're in stable.

What's your opinion @luceos?

@Eldenroot
Copy link
Author

This has been the behavior since the earliest beta's. If we want to change this we need to Approve it first. Please use https://discuss.flarum.org/t/proposals to file it for consideration.

This is funny :) Lets talk about a bug and make a poll if we can consider it as a bug :D

@luceos
Copy link
Member

luceos commented Sep 16, 2022

I am trying to follow the procedure as laid out here. I can see that in this situation I have been too inconsiderate.

@luceos luceos reopened this Sep 16, 2022
@Eldenroot
Copy link
Author

Did you follow EU politics with this protocol? :D Why make things easy if there is a long and complicated shortcut :) Nevermind, it is up to you... I did my job, now it is your turn... do your work, keep Flarum alive :)

@SychO9
Copy link
Member

SychO9 commented Sep 16, 2022

@luceos has reopened the issue, there is no need for any unnecessary comments. Thank you.

@SychO9 SychO9 self-assigned this Apr 15, 2023
@SychO9 SychO9 added this to the 1.8 milestone Apr 15, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants