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

Use DisabledComposerView instead of no composer to add message #54

Merged
merged 4 commits into from
Aug 31, 2022

Conversation

MadLittleMods
Copy link
Contributor

@MadLittleMods MadLittleMods commented Aug 30, 2022

Use DisabledComposerView instead of no composer to add message:

You're viewing an archive of events from 2022-02-08. Use a Matrix client to start chatting in this room.

Archive with message at the bottom


Also refactors some of the views and viewmodels out to their own files

Todo

MadLittleMods added a commit to element-hq/hydrogen-web that referenced this pull request Aug 30, 2022
@MadLittleMods MadLittleMods added the T-Enhancement New feature or request label Aug 30, 2022
@MadLittleMods MadLittleMods marked this pull request as ready for review August 31, 2022 00:10
@MadLittleMods MadLittleMods merged commit d9c0f66 into main Aug 31, 2022
MadLittleMods added a commit to element-hq/hydrogen-web that referenced this pull request Aug 31, 2022
MadLittleMods added a commit that referenced this pull request Aug 31, 2022
…changes

We added the description in #54
but then made a fix to one of the after render errors, #57,
which exposed that the description is then overriden when the powerlevel changes.

We now just override the getters directly instead of the internal fields
which are overwritten internally.
MadLittleMods added a commit that referenced this pull request Aug 31, 2022
…changes (#58)

We added the description in #54 but then made a fix to one of the after render errors, #57, which exposed that the description is then overriden when the powerlevel changes.

We now just override the getters directly instead of the internal fields which are overwritten internally.

---

This also fixes a warning from Hydrogen in the console:

```
disposable not found, did it leak? {kind: 'disabled', description: Array(3)}
```
@MadLittleMods MadLittleMods added the A-archive-room-view The view to look at a room day by day in the archive label Apr 11, 2023
@MadLittleMods MadLittleMods deleted the madlittlemods/use-disabled-composer-view branch April 26, 2023 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-archive-room-view The view to look at a room day by day in the archive T-Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant