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

Bare HTML support #16

Closed
wants to merge 13 commits into from
Closed

Bare HTML support #16

wants to merge 13 commits into from

Conversation

noxware
Copy link
Owner

@noxware noxware commented Mar 7, 2024

Closes #11

@eftychis
Copy link

Interesting. If I understand correctly the main change is keeping track of egui ids and allowing a null id.

Do you have any thoughts of merging and releasing this change @noxware? Or is there a problem?

@noxware
Copy link
Owner Author

noxware commented May 10, 2024

@eftychis Yeah, a change to how each area is identified and tracked was needed to support HTML outside of a window.
I think the main reasons I didn't merge this were:

  • I wanted to see if I can make this work inside scroll views.
  • I needed to do some manual regression testing to ensure my changes didn't break the existing examples.
  • I wanted to experiment a bit with this change to see if this approach can be used for more complex stuff and identify possible edge cases.

Ahhh yeah, testing it quickly I remember there was a problem with the focus in the following screenshot:

image

You can't select text in the right in the markdown editor example. That's something to fix.

Sorry if you were interested in this and I couldn't complete it yet 😞 I will probably resume it some day, but in these last months my free time (that I also use to code other personal projects) have been very valuable and this become a bottom priority for now. Of course I'm open to contributions if you want to make a PR completing this sooner.

@noxware
Copy link
Owner Author

noxware commented Oct 30, 2024

Update: Sometimes I touch this project and I drop some lines into the next branch. There I was exploring many ideas like:

  • An overall simplification of the code base.
  • Experimenting on making browser support more consistent (if possible).
  • And supporting bare html elements like the ones in this PR.

This project is still something I touch very rarely but small progress is still progress. Anyways, I may close this PR as this feature may come from that other branch eventually.

@noxware noxware closed this Oct 30, 2024
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.

Support bare HTML without a window
2 participants