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

File dialog listbox overlaps with buttons #3967

Open
tznind opened this issue Mar 8, 2025 · 5 comments
Open

File dialog listbox overlaps with buttons #3967

tznind opened this issue Mar 8, 2025 · 5 comments
Labels
bug v2 For discussions, issues, etc... relavant for v2

Comments

@tznind
Copy link
Collaborator

tznind commented Mar 8, 2025

Describe the bug
Listbox entries go too far down, should only go to buttons

Screenshots
Image

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

Set Project & Milestone
If you have access, please don't forget to set the right Project and Milestone.

@tznind tznind added bug v2 For discussions, issues, etc... relavant for v2 labels Mar 8, 2025
@tznind
Copy link
Collaborator Author

tznind commented Mar 8, 2025

Hmn this actually seems to only be in TGD, it is working in UICatalog...

Image

@tznind
Copy link
Collaborator Author

tznind commented Mar 16, 2025

Ok I can now repro, fix is to add a call to Layout before using e.g.

var ofd = new OpenDialog()
{
    Title = "Open",
    AllowedTypes = new List<IAllowedType>(new[] { new AllowedType("View", SourceCodeFile.ExpectedExtension) })
};
ofd.SetupNiceColorSchemes();

// This call fixes issue
ofd.Layout();

Application.Run(ofd, this.ErrorHandler);

@BDisp
Copy link
Collaborator

BDisp commented Mar 16, 2025

Maybe the fix is to call Layout(); inside the SetupNiceColorSchemes(); method.

@tznind
Copy link
Collaborator Author

tznind commented Mar 16, 2025

Maybe v2 driver run should Layout on Run, it might also be the reason for #3951

@BDisp
Copy link
Collaborator

BDisp commented Mar 16, 2025

Maybe v2 driver run should Layout on Run, it might also be the reason for #3951

I think it's better on Begin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug v2 For discussions, issues, etc... relavant for v2
Projects
None yet
Development

No branches or pull requests

2 participants