Falko Schindler

Results 231 comments of Falko Schindler

I think it's better to create a separate PR for Monaco, because it is a separate feature after all and might get merged before or after the CodeMirror element (or...

Thanks for the pull request, @merlinarer! In order to keep it simple and more close to the original example, I updated the latter with the idea of incrementally building the...

Ok, thanks, @CrystalWindSnake! Just let me known when you think it's a good time for a first code review. 🙂

@CrystalWindSnake Issue #3939 just reminded me of this PR. Are you still working on it? Is there anything we can do to help? Your last [update](https://github.com/zauberzeug/nicegui/pull/3261#issuecomment-2347246081) sounded very promising.

Thanks for reporting this issue, @MuuXB! When running with an existing FastAPI app, there seems to be no `Server.instance`. We should either find a reference to the uvicorn server or...

@MuuXB Why did you close this issue? As far as I can tell, the problem still persists and we're still looking for a way to fix it.

Hi @ghbm-itk, this is an unfortunate consequence of how NiceGUI adds default padding to `ui.card`, which Quasar doesn't. It has been discussed in #726 and https://github.com/zauberzeug/nicegui/discussions/1295#discussioncomment-6588788 and is documented [here](https://nicegui.io/documentation/card#the_issue_with_nested_borders).

@ghbm-itk I don't think it's a good idea to automatically change the element hierarchy, because it would break certain assumptions (see #2040 for a discussion about a similar case). I...

@ghbm-itk Sorry for the late reply. I think it's a bit different: - QCards usually don't have any padding. But you can add padding with QCardSections: ```py with ui.element('q-card'): ui.label('No...

I just closed PR #2301, because we're no longer convinced that it is a good idea to add additional layers to `ui.card` to fix a layout inconsistency between NiceGUI and...