View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0002117NoesisGUIC# SDKpublic2021-09-17 16:252021-09-30 18:54
Reporterai_enabled Assigned Tosfernandez  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.1.1 
Target Version3.1.2Fixed in Version3.1.2 
Summary0002117: Reloading the texture (via RaiseTextureChanged) which is used in tiled ImageBrush doesn't recalculate the brush tiling
DescriptionHi guys,
there is a new bug (since v3.0 or maybe even v3.1.0 as I didn't try it) with texture reloading for ImageBrush.

Example XAML:
    <ImageBrush x:Key="WidgetBackgroundImageTexture"
                ImageSource="WidgetBackground.png"
                TileMode="Tile"
                Stretch="Uniform"
                Viewport="0,0,5.5,5.5"
                ViewportUnits="Absolute" />


See the attached WidgetBackground.png file and screenshots A and B.

Screenshot A: the texture is loaded immediately and RaiseTextureChanged is never called. Proper appearance.

Screenshot B: the texture is not loaded immediately. How it's loaded by our custom TextureProvider (worked fine in v3.0):
- TextureProvider via GetTextureInfo method provides actual texture dimensions
- TextureProvider via LoadTexture method provides null result (as the texture is not yet loaded) and starts loading the texture asynchronously
- once the texture is loaded, RaiseTextureChanged is invoked

After the fix in https://www.noesisengine.com/bugs/view.php?id=2115 RaiseTextureChanged does work but there is this bug with the ImageBrush not being updated.


Besides, I've noticed a difference in how the tiling works in 3.1 compared to 3.0 (for now I'm using instant loading as it doesn't causes any loading bugs). I have to modify all the cases where we're using tiling brushes (at least by modifying Viewport) but the tiling behavior is still visually slightly different compared to what it was before. I see there are no changes in v3.1 release notes regarding this. Is this an expected change? What exactly was changed? I'm really confused regarding this.

Regards!
TagsNo tags attached.
PlatformAny

Activities

ai_enabled

ai_enabled

2021-09-17 16:25

updater  

WidgetBackground.png (1,127 bytes)   
WidgetBackground.png (1,127 bytes)   
A_Proper.png (1,089,238 bytes)
B_Issue.png (1,093,056 bytes)
ai_enabled

ai_enabled

2021-09-17 18:39

updater   ~0007456

Regarding the ImageBrush tiling. I've noticed that there is no differences when using Viewport+ViewportUnits in 3.0.11 vs 3.1.1 however something has definitely changed regarding the Viewbox and ViewboxUnits properties and how they applied.
sfernandez

sfernandez

2021-09-20 11:03

manager   ~0007458

Last edited: 2021-09-20 11:03

 Hi, I'm going to investigate this reloading issue and let you know.
Regarding the tiling, in 3.1 we implemented all Tile modes to properly work in atlas textures. So if you were using a subrect of the source image, it will behave now different from 3.0 version. In 3.1 all tiling modes should look like in WPF.
sfernandez

sfernandez

2021-09-20 16:33

manager   ~0007462

I found that hot-reloding is not affecting frozen images, we need to think how to solve that properly.

Apart from that, I noticed that you defined the background image as follows:
<ImageBrush x:Key="WidgetBackgroundImageTexture"
                ImageSource="/UI/Images/Backgrounds/WidgetBackground.png"
                TileMode="Tile"
                Stretch="Uniform"
                Viewport="0,0,8,8"
                Viewbox="0,0,12,12"
                ViewboxUnits="Absolute"
                ViewportUnits="Absolute" />

The Viewbox is bigger than Image size, which is 8x8, so this will leave en empty space between tiles now (according to WPF behavior). This is probably what you are seeing different from 3.0, where viewbox bigger than image was ignored.
ai_enabled

ai_enabled

2021-09-20 16:46

updater   ~0007463

Hi Sergio,
thank you for your explanation. Regarding the Viewbox—yes, I was using it this way with the previous NoesisGUI version to get the desired look (for some reason it was the only approach that worked fine). I've removed Viewbox property altogether and fine-tuned Viewport instead—seems good now.

Regarding the frozen images—I hope you can find a useful approach soon. Good luck!
sfernandez

sfernandez

2021-09-20 18:24

manager   ~0007464

I fixed the reloading issue, please download a patched library here:
https://drive.google.com/file/d/1-JUB0k1nLyJIa4c9hZdrpMkl6VDXsDXA/view?usp=sharing
sfernandez

sfernandez

2021-09-30 18:47

manager   ~0007493

Did you have a chance to try the patched library?
ai_enabled

ai_enabled

2021-09-30 18:51

updater   ~0007494

Last edited: 2021-09-30 18:51

 Hi Sergio,
I'm not sure but the comment was not posted here, perhaps a cookie was outdated (that happens often!) and I didn't check after posting whether it was actually posted.
I've replied on the same date—I've tried the patched library and the issue is resolved. Please close this ticket!
The only blocking issue preventing us from using 3.1 now is https://www.noesisengine.com/bugs/view.php?id=2121

Regards!
sfernandez

sfernandez

2021-09-30 18:54

manager   ~0007495

Thanks, that ticket was the next thing we wanted to resolve.

Issue History

Date Modified Username Field Change
2021-09-17 16:25 ai_enabled New Issue
2021-09-17 16:25 ai_enabled File Added: WidgetBackground.png
2021-09-17 16:25 ai_enabled File Added: A_Proper.png
2021-09-17 16:25 ai_enabled File Added: B_Issue.png
2021-09-17 16:25 ai_enabled Description Updated
2021-09-17 18:39 ai_enabled Note Added: 0007456
2021-09-20 11:03 sfernandez Assigned To => sfernandez
2021-09-20 11:03 sfernandez Status new => feedback
2021-09-20 11:03 sfernandez Note Added: 0007458
2021-09-20 11:03 sfernandez Note Edited: 0007458
2021-09-20 11:03 sfernandez Target Version => 3.1.2
2021-09-20 16:33 sfernandez Note Added: 0007462
2021-09-20 16:46 ai_enabled Note Added: 0007463
2021-09-20 16:46 ai_enabled Status feedback => assigned
2021-09-20 18:24 sfernandez Status assigned => feedback
2021-09-20 18:24 sfernandez Note Added: 0007464
2021-09-30 18:47 sfernandez Note Added: 0007493
2021-09-30 18:51 ai_enabled Note Added: 0007494
2021-09-30 18:51 ai_enabled Status feedback => assigned
2021-09-30 18:51 ai_enabled Note Edited: 0007494
2021-09-30 18:54 sfernandez Status assigned => resolved
2021-09-30 18:54 sfernandez Resolution open => fixed
2021-09-30 18:54 sfernandez Fixed in Version => 3.1.2
2021-09-30 18:54 sfernandez Note Added: 0007495