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

Bug: UI buttons on Behaviors, Behavior sets, Archetypes and Scenarios type almost not accessible. (Win10) #41

Open
Goutye opened this issue Jan 24, 2018 · 21 comments
Assignees
Labels

Comments

@Goutye
Copy link

Goutye commented Jan 24, 2018

As demonstrated in the image below:
image

The button to create Behavior/Archetype/Scenario is almost not accessible.
Also, the Considerations UI inside Behaviors Tab (Edit Consideration) is not visible.
Something similar happens in the Behavior set tab (right part)

Changing the size of the window does not solve the problem.

@apoch apoch self-assigned this Jan 24, 2018
@apoch apoch added the bug label Jan 24, 2018
@apoch
Copy link
Owner

apoch commented Jan 24, 2018

Hi, thanks for the report. I'm having trouble reproducing the issue. Can you check your display scaling settings for me and post the current values?

  • Right click desktop
  • Choose Display Settings
  • Scroll down to Scale and Layout

I'm testing the UI at 100% scaling. Is there a chance you're using a different scaling/DPI configuration? I can also experiment with fonts as it looks like your system font is different than mine.

Thanks, just trying to pin down what's going wrong here!

@apoch apoch added this to Backlog in GDC 2018 Release Jan 24, 2018
@Goutye
Copy link
Author

Goutye commented Jan 24, 2018

Hi. Here are my current values:
100% (Recommended)
1920x1200

I don't remember having change any display setting on my Windows version.
Despite working in Japan, this Windows 10 version is EN(US) (Language pack EN(US)).

I will try to check tonight on my home computer.

Edit: Never tried to build UI in C# yet, but I have some doubts on the behavior of the following lines:

this.InputFlowPanel.Location = new System.Drawing.Point(6, 19);
this.InputFlowPanel.Size = new System.Drawing.Size(400, 135);

It seems to be possible to set a margin instead of drawing from a specific relative location (If that was the intended behavior of the above lines).
As for the Size, it should be adjusted automatically I guess.
Anyway, no great experience on C# UI yet, I just read the doc and code quickly, so I may misunderstand part of the code.

@apoch
Copy link
Owner

apoch commented Jan 25, 2018

I think that specific code is from this file which is an automatically generated module that sets up the default properties of the UI elements. The Designer.cs modules are all similarly generated by the UI editor from Visual Studio.

In that particular case the panel is "attached" to its parent control and should resize with it automatically (see some lines up). Almost all of the UI elements do something similar. WinForms is kind of gross in the way it handles automatic resizing, so this sort of silliness is common.

What bothers me is that this seems to be very machine-specific. I'm not seeing the incorrect sizing behavior across three different Win10 machines, and I know a few others have downloaded the beta and have not reported any trouble.

So I am naturally scared that this will go wrong for an unpredictable number of people :-/

Unfortunately without a reproducible case it's really hard to even speculate on what might be broken.

@Goutye
Copy link
Author

Goutye commented Mar 26, 2018

Sorry for the late follow up.

I tried the latest release. while I still have some UI bugs, an annoying one (hiding totally the buttons of EditConsiderations) is now partially fixed (still almost fully hidden). The scenario buttons are still totally hidden, which prevent any simulation.

image

I tried the application on a similar PC (Win10), the bug did not appear. I guess, if no one else reports this bug, you can consider it unique to my setup. I have another computer to enjoy the application, so it is fine.

@apoch apoch removed this from Backlog in GDC 2018 Release Jul 24, 2018
@apoch apoch added this to Backlog in Curvature 2.0 Release Jul 24, 2018
@taoshang
Copy link

I also have this bug in Win10.

@apoch
Copy link
Owner

apoch commented Sep 30, 2018

@taoshang I am curious if you see this in a release package of Curvature, or in the latest source code version, or possibly both?

@taoshang
Copy link

taoshang commented Oct 9, 2018

@apoch I only try this is latest source code version.

@taoshang
Copy link

taoshang commented Oct 9, 2018

@apoch Sorry, I only try this in "Download the master branch" version.

@taoshang
Copy link

taoshang commented Oct 9, 2018

@apoch I have tested both version. I see this in both.

@apoch
Copy link
Owner

apoch commented Oct 15, 2018

@taoshang Sent you a pull request since you have a fork of the repository available. It adds some diagnostic stuff to try and track this down. If you can think of any other useful information that could be collected, feel free to add it in.

taoshang added a commit to taoshang/curvature that referenced this issue Oct 16, 2018
Add some diagnostic stuff to track down UI bug apoch#41
@taoshang
Copy link

@apoch
Window font face: 宋体
Window font size: 9
Autoscale factor X: 1
Autoscale factor Y: 1
Autoscale basis X: 6
Autoscale basis Y: 12
Autoscale current X: 6
Autoscale current Y: 12
Scenarios Autoscale basis X: 6
Scenarios Autoscale basis Y: 12
Scenarios Autoscale current X: 6
Scenarios Autoscale current Y: 12

2018_10_16_15_57 50

@taoshang
Copy link

Window font face: 宋体
Window font size: 9
Autoscale factor X: 1
Autoscale factor Y: 1
Autoscale basis X: 6
Autoscale basis Y: 12
Autoscale current X: 6
Autoscale current Y: 12
Scenarios Autoscale basis X: 6
Scenarios Autoscale basis Y: 12
Scenarios Autoscale current X: 6
Scenarios Autoscale current Y: 12

2018_10_16_15_58 03

@taoshang
Copy link

Microsoft Visual Studio Community 2017 Version: 15.7.6
Microsoft .NET Framework Version: 4.7.03056
Windows 10 x64

@taoshang
Copy link

taoshang commented Oct 16, 2018

if I reduce the font size in UI designer, the button will show.
I guess I don't have the font you use. Could you tell me what's font in the UI designer?
I can download it and try again.

@apoch
Copy link
Owner

apoch commented Oct 17, 2018

Here's my settings:

Window font face: Microsoft Sans Serif Window font size: 8.25 Autoscale factor X: 1 Autoscale factor Y: 1 Autoscale basis X: 6 Autoscale basis Y: 13 Autoscale current X: 6 Autoscale current Y: 13 Scenarios Autoscale basis X: 6 Scenarios Autoscale basis Y: 13 Scenarios Autoscale current X: 6 Scenarios Autoscale current Y: 13

I think the problem might be that autoscale is not correctly set for the UserControls in the project. That would make sense with the difference in Y scale (12 vs 13) and the font size change you found. I will open another PR in a moment with an attempt at setting the autoscale properties the right way.

@taoshang
Copy link

@apoch the new merge has fixed most of UI layout bugs.
But there is a bug here:
2018_10_18_10_10 19

@taoshang
Copy link

@apoch the down arrow button is still missing of response curve designer in right bottom.

@apoch
Copy link
Owner

apoch commented Oct 19, 2018

@taoshang Interesting. I'm not sure why that element is different yet.

On that page from the screenshot, if you click the Wizard... button, there is a dialog that also uses the response curve designer UI - it's under the Apply Response Curve tab along the top, which has an Advanced mode on the left side. Does that response curve UI also have the down arrow missing?

Just trying to narrow down what Winforms behavior might be involved. Thanks for the screenshots and merges, it has been very helpful!

@taoshang
Copy link

2018_10_19_14_42 25
2018_10_19_14_43 15

@taoshang
Copy link

@apoch
the response curve designer UI in "edit considerations" have the down arrow missing. no matter how I change the size of the window.
the response curve designer UI in Wizard show all buttons.
Thank you.

@apoch
Copy link
Owner

apoch commented Oct 20, 2018

@taoshang I changed the placement of the down arrow button (and the height of the side buttons) to be manually controlled in code instead of by WinForms. The Wizard dialog and the main response curve designer should now look identical with this change. Will open a PR shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

No branches or pull requests

3 participants