-
Notifications
You must be signed in to change notification settings - Fork 174
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
min text area width? #61
Comments
I think here bevy-inspector-egui/src/impls/std.rs Line 17 in 3190cbc
we need to set be able to set https://github.com/emilk/egui/blob/87ca29173d2d5dd5421b80b274fa59504757918f/egui/src/widgets/text_edit/builder.rs#L66 |
It may be necessary but it's not sufficient. I think something in the parent component needs to change too. |
Not using derive inspectable but implementing it myself on the parent was key: ui.vertical_centered(|ui| {
Grid::new(context.id())
.min_col_width(500.)
.show(ui, |ui| { Happy. |
I believe this is caused by not setting num_columns on the Grid - https://docs.rs/egui/latest/src/egui/grid.rs.html#305-308 - so layout has no idea whether it needs to leave space for more columns. Setting num_columns to 2 and then wrapping the field value inspector in ui.allocate_ui or whatever should make everything work much more sensibly. |
Notice how bevy-inspector-egui/crates/bevy-inspector-egui/src/reflect_inspector/mod.rs Lines 462 to 469 in ae7cdf5
fn ui_for_struct_readonly(
&mut self,
value: &dyn Struct,
ui: &mut egui::Ui,
id: egui::Id,
options: &dyn Any,
) {
Grid::new(id)
+ .num_columns(2)
.show(ui, |ui| { Probably will want to add |
Is there a way we can force it to not have such narrow text areas? At the moment it seems to wrap at a line length of 7 chars or so.
I tried the above but it seemed to make no effect. Other things are wider so it could use up more space if it wanted to but it chooses not to. Being able to set min / max text width as an attribute would be good.
The text was updated successfully, but these errors were encountered: