Skip to content

feat: Element Call#2536

Closed
hanthor wants to merge 4 commits into
cinnyapp:devfrom
hanthor:dev
Closed

feat: Element Call#2536
hanthor wants to merge 4 commits into
cinnyapp:devfrom
hanthor:dev

Conversation

@hanthor

@hanthor hanthor commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Description

Note: this is very much AI generated and not yet thoroughly tested

This implements the widget API inasmuch to get embedded Element Call to work. I'll probably maintain this as a fork, but I'm submitting a PR in case there's any desire for it

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions

github-actions Bot commented Nov 19, 2025

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@hanthor hanthor marked this pull request as draft November 19, 2025 14:15
@hanthor

hanthor commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

ajbura added a commit to cinnyapp/cla that referenced this pull request Nov 19, 2025
@hanthor

hanthor commented Nov 19, 2025

Copy link
Copy Markdown
Contributor Author

This works (barely). I know another implementation is in the works. But I want to keep recommending Cinny to people and the lack of calling is a deal breaker to many

Comment thread package.json
"linkify-react": "4.1.3",
"linkifyjs": "4.1.3",
"matrix-js-sdk": "38.2.0",
"matrix-js-sdk": "34.11.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it intended that you seem to be downgrading matrix-js-sdk here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No this was just me copy pasting stuff over from element web to get it to work. It can get updated.

The widget box needs a lot of improvement too. It could have a picture in picture and a fullscreen. I think fullscreen is necessary

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I would agree. I have also been looking at #2512 (which looked promising) and having not much luck running that one either. :/

I'd like to use cinny but the lack of video calling is a major blocker.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you looked at #2335 I'm currently using it and it seems to work well enough why is there three different versions of this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because I didn't look at the open PRs

@kfiven

kfiven commented Feb 14, 2026

Copy link
Copy Markdown
Member

Closing in favor of #2599

@kfiven kfiven closed this Feb 14, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants