User forums > Nightly builds
The 20 February 2011 build (7017) is out.
Jenna:
--- Quote from: jens on March 08, 2011, 10:56:16 pm ---The automake-stuff will come with my next patch, it was not on top of my priority list.
--- End quote ---
Here it comes.
Some small changes :
focus-rect nowalso includes bitmap and close-icon as done by gtk,
close-button removed from DraSimpleButtun, because it's not used there.
A real fix: texth was used to calculate texty before it was calculated :oops:
Curiously this issue only leads to an incorrect text-placement in the first tab, if it is not the active tab.
oBFusCATed:
But my comment is still regarded as spam...
Jenna:
--- Quote from: oBFusCATed on March 09, 2011, 10:29:27 am ---But my comment is still regarded as spam...
--- End quote ---
Some of my posts are also regarde as spam, especially if they contain many quotes.
Yours is approved now.
If that happens you can send me or another admin a PM.
--- Quote from: oBFusCATed on March 08, 2011, 11:26:44 pm ---Okay, you've broken my theme -> http://smrt.is-a-geek.org/codeblocks/screens/cb_gtk_tool_breakage.png
On the second row is the working version with my patch, forth row is screenshot from pidgin.
--- Quote from: jens on March 08, 2011, 10:56:16 pm ---I attach two diffs, one for the header and one for the cpp file.
The diffs compare only the relevant part of the notebokstyles.{h|cpp} files with your files.
--- End quote ---
There are many irrelevant changes, so it is hard to follow :(
--- Quote from: jens on March 08, 2011, 10:56:16 pm ---The main changes:
1.I removed GetIcon() and m_close_icon completely, because wxGTK seems not to update the "gtk-close" icon if the theme is changed, only on program startup.[/li][/list]
2.I fetch the actual close-icon from stock and rescale it if necessary to the wanted size (16x16)[/li][/list]
3.The wanted close-icon-size is set as static variable, I also added a setter-function to change the size, but it is actually not used[/li][/list]
4.I removed completely GetTabSize[/li][/list]
5.I use m_normal_font as measuring-font in GetBestTabCtrlSize()[/li][/list]
6.I always use normal font (for all tabs as gtk seems to do it), a better approach would surely be to get the actual notebook-font from gtk if possible[/li][/list]
7.I changed ArrowStateAndShadow() to return values for disabled, hovered, pressed and normal buttons, according to the way it is done in by gtk[/li][/list]
8.I use nullptr instead of NULL, wherever it is possible[/li][/list]
9.I overwork DrawTab() at many places to create tabs with different height for normal and selected tabs, to fetch the actual close-button from gtk-stock, to draw a focus-rect and to place the bitmap and the label[/li]
[/list]
--- End quote ---
1. Probably this should be fixed in the wxGTK, not workarounded?
4. Why?
5. Why, too?
8. Don't do it please, the main idea was to prepare this for inclusion in wxGTK, not distributed in C::B
9. Seems broken with my theme :( -> Darkilouche
--- Quote from: jens on March 08, 2011, 10:56:16 pm ---To fix the issue with not correctly applied/used TabArtProvider, I use target->SetTabCtrlHeight(0); before applying the new ArtProvider and target->SetTabCtrlHeight(-1); after applying it. Doing so seems to fix the issue (at least here).
--- End quote ---
This seems to work, probably you can apply it separately...
--- Quote from: jens on March 08, 2011, 10:56:16 pm ---The automake-stuff will come with my next patch, it was not on top of my priority list.
I do not use your GTK2 test from configure.in, because we already test for it some lines later, I just remove the if-xlause around the test, as we need it for C::B's core now.
--- End quote ---
OK, my auto-fu is very low...
--- End quote ---
1. Surely, but it is not yet fixed and it might take a while until this happens (probably never). Sometimes it takes very long until fixes like this will be accepted or it never happens, but it is aminor issue, because changing the icon-theme while C::B is running will not happen so often.
4. Because there is no need to derive it.
5. because of 6. The measuring-font is by default m_selected_font and it is a bold-font, but as far as I have seen all gtk-themes use normal fonts for the tab-labels, if a bold-font is used for measuring, but a normal one is efectively used, the tab-size is too large for the active tab.
8. Just replace all nullptr's with NULL and it is done. I think it's a great enhancement for C::B to use the "real" gtk-theme for the notebooks (if selected) and until it is in wxWidgets, I like to have it in C::B (and as implied in 1. it sometimes take very much time until new things make it into wxWidgets). And I prefer to use nullptr instead of NULL for uninitialized pointers.
9. Should be fixed with the new patch.
The broken theme should be fixed with the new patch (incorrect calculation of texty).
oBFusCATed:
BTW: the forum started to behave strangely... missing posts, when using the 'Show unread post since ...', so someone have changed some setting, which broke it...
1. Why don't we leave it just like it is now and report a problem to wxGTK. It is not a very big problem after all.
8. I would prefer both versions to be very similar, because if this is added to wxGTK it will be added at trunk,
so wx2.8 won't have it and we would have to support it in C::B.
If the versions are very different it would be harder to keep them in sync.
I'll test your new patch tonight and report if there are problems.
p.s. probably you should add some copyright names in the license header, so if the code would be included in wxGTK, a license change would be required and as this is a team effort now, all contributors should approve it (I'm not a lawyer, so I could be wrong).
oBFusCATed:
Jens:
The new patch doesn't fix my theme...
Probably, you've calculated too large height of the tab.
The nice edge at the bottom is missing, see the screen shot I've made...
This border is missing from many themes I have on my machine
(my themes are old, because I've not updated theme for the last 6-12 months, but I doubt it makes a difference)
Some themes are even more broken. This is because they want to make the notebook higher that possible (probably) in wxAuiNotebook.
Probably that is why I've not done the selected tab is higher thing. Firefox does the same -> all tabs have the same height...
One very broken theme is "Milke 2.3"...
The focus rectangle is very broken in the "MurrinaAqualsh" theme....
On all themes the text is moving in the wrong direction (up instead of down), when the tab is unselected.
On the other hand, your patch does many things well...
The fonts are always working, which my patch does not do successfully.
The close button is placed, correctly, for all themes, my patch failed on some of them...
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version