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

Mark as unread #2

Merged
merged 18 commits into from
Nov 19, 2021
Merged

Mark as unread #2

merged 18 commits into from
Nov 19, 2021

Conversation

su-ex
Copy link
Member

@su-ex su-ex commented Nov 19, 2021

Closes SchildiChat/schildichat-desktop#109


This PR currently has no changelog labels, so will not be included in changelogs.

Add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is plus X-Breaking-Change if it's a breaking change.

alangecker and others added 18 commits July 4, 2021 18:33
Signed-off-by: chandi <git@chandi.it>
Signed-off-by: chandi <git@chandi.it>
Signed-off-by: chandi <git@chandi.it>
Signed-off-by: chandi <git@chandi.it>
Conflicts:
	src/components/structures/TimelinePanel.tsx
	src/components/views/rooms/RoomTile.tsx
	src/stores/notifications/RoomNotificationState.ts
From Google's material design icons:
https://fonts.google.com/icons?selected=Material+Icons&icon.query=eye

License: https://google.github.io/material-design-icons/
"We have made these icons available for you to incorporate them into
your products under the Apache License Version 2.0. Feel free to remix
and re-share these icons and documentation in your products. We'd love
attribution in your app's about screen, but it's not required. The only
thing we ask is that you not re-sell the icons themselves."
TypeError: Cannot read property 'getAccountData' of null
    at d (Rooms.ts:160)
    at e.removeUnreadMarker (TimelinePanel.tsx:884)
    at e.<anonymous> (TimelinePanel.tsx:1140)
    at ui (react-dom.production.min.js:131)
    at ps (react-dom.production.min.js:220)
    at wc (react-dom.production.min.js:259)
    at t.unstable_runWithPriority (scheduler.production.min.js:18)
    at jo (react-dom.production.min.js:122)
    at xc (react-dom.production.min.js:252)
    at pc (react-dom.production.min.js:243)

when you have the panel for mentions open.
This can get in the way of the "marked as unread" icon otherwise.
Furthermore, I didn't really like it showing there persistently either
way.
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.

Mark as unread
3 participants