Bug #6293
openRegression in 3.3.9 for bug #1947
0%
Description
I posted an issue about 5 years ago that was fixed, but I see the original behavior in 3.3.9 now. The issue number is 1947 and includes a test case.
Files
Updated by Velvet Jones over 6 years ago
Updated by Roel Standaert over 6 years ago
- File issue_6293.patch issue_6293.patch added
So I have a patch that seems to improve things, but I'm not entirely convinced that it's the appropriate solution...
Updated by Velvet Jones over 6 years ago
I've tried the patch, it works - sorta.
My code looks like this:
this~~m_layout>setColumnStretch(0,0);m_layout
this>setColumnStretch(1,this->m_columnCount > 2 ? 60 : 75);m_layout~~>setColumnStretch(2,this->m_columnCount > 2 ? 40 : 25);
this
For 3 columns it looks right, but for 2 columns I seem to get 50/50. Tried changing to 80/20 in the code above for 3-columns, but it still shows the exact same size as 75/25.
Updated by Velvet Jones over 6 years ago
Correction: I tried changing to 80/20 for **2* columns, not 3 columns. Result still appears to be a 50/50 split when set to 80/20.
Updated by Roel Standaert over 6 years ago
- Status changed from New to Feedback
Could you provide some example code for that? When I change your initial example code it seems fine.
Updated by Velvet Jones over 6 years ago
Try this:
Using the existing test harness, simply click on the "Test" tab repeatedly. On my system, the column widths change every time until they "settle".
Updated by Roel Standaert over 6 years ago
- Target version changed from 3.3.10 to 3.3.11
Updated by Velvet Jones over 6 years ago
Gotta say, I would be eternally grateful if you could get this fixed in 3.3.10. I've been putting off a release because this affects the layout of almost every page in my app.
Updated by Roel Standaert over 6 years ago
I just released 3.3.10. I really couldn't put off its release any longer. I will look further into this, but it should be possible to just apply the fix as a single patch on top of 3.3.10.
Updated by Velvet Jones over 6 years ago
When I apply the patch to the 3.3.10 release, my sub-menus no longer pop up. And then of course it also shows the original behavior with the grid column widths being shifted every time I click the tab, etc.
Updated by Velvet Jones over 6 years ago
Any chance I can get a fix for this problem?
Updated by Koen Deforche over 6 years ago
- File fix-6293.patch fix-6293.patch added
Hey,
Can you try attached patch?
Regards,
koen
Updated by Velvet Jones over 6 years ago
I've tried it, it's close. Instead of updating the test case I have attached a screenshot showing the result behavior after applying the patch. I am simply clicking between multiple tabs, and the screen layout keeps adjusting as I click different tabs. So I end up chasing the tabs around the screen in order to switch between them.
Updated by Koen Deforche over 6 years ago
Hey,
Did you test with or without Roel's patch? Preferably I'ld like to know the result without Roel's patch?
If it still fails: it would be really helpful if the test cases reproduces the issue.
Koen
Updated by Velvet Jones over 6 years ago
I tested without Roel's patch. A git diff shows that the only files changed from the detached head tag 3.3.10 are 'src/js/WtResize.js' and 'src/js/WtResize.min.js'.
I'll add a tab and some various data to the tabs to see if I can get the test case to reproduce.
Updated by Velvet Jones over 6 years ago
I've added a test case that seems to reproduce the problem. When you alternate between clicking the two tabs, the layout shifts each time.
Updated by Roel Standaert about 6 years ago
- Target version changed from 3.3.11 to 3.3.12
Updated by Roel Standaert almost 6 years ago
- Target version changed from 3.3.12 to 3.4.0