-
Notifications
You must be signed in to change notification settings - Fork 2
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
Conversation
this was causing the unwrap in line 101 to blow up but setting the value to empty string is no longer necessary (i think), removing it should work
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? |
@eftychis Yeah, a change to how each area is identified and tracked was needed to support HTML outside of a window.
Ahhh yeah, testing it quickly I remember there was a problem with the focus in the following screenshot: 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. |
Update: Sometimes I touch this project and I drop some lines into the
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. |
Closes #11