View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
0001720 | OpenClonk | Engine - Rendering | public | 2016-04-17 21:05 | 2016-04-21 19:21 | ||||||||
Reporter | Maikel | ||||||||||||
Assigned To | |||||||||||||
Priority | high | Severity | minor | Reproducibility | always | ||||||||
Status | new | Resolution | open | ||||||||||
Product Version | git master | ||||||||||||
Target Version | git master | Fixed in Version | |||||||||||
Summary | 0001720: SetGraphics with GFXOV_MODE_Object takes both visibility of parent object and overlay_object into account | ||||||||||||
Description | The one of overlay_object should not be taken into account, because that way you can never control the visibility of the new graphics independently. | ||||||||||||
Tags | No tags attached. | ||||||||||||
Attached Files |
|
![]() |
|
Maikel (developer) 2016-04-17 21:08 |
Reminder sent to: Zapper |
Maikel (developer) 2016-04-17 21:08 |
Reminder sent to: Sven2 |
Sven2 (developer) 2016-04-18 15:46 Last edited: 2016-04-18 15:46 |
Can't you just attach a helper object to the clonk (make sure it doesn't take shockwave energy!) and overlay that with the modified graphics? I'm not sure I like that it's not enough to set an object's visibility, but you also have to know about all its overlays. |
Maikel (developer) 2016-04-18 18:04 |
That is what I did, I attach object dummy to the clonk and use: clonk.Visibility = VIS_Owner; dummy->SetGraphics(nil, nil, GFX_Overlay, GFXOV_MODE_Object, nil, nil, clonk); dummy.Visibility = VIS_All; Then the dummy is only visible to the owner of the clonk, i.e. the combination of the two visibilities. |
Marky (developer) 2016-04-21 19:21 |
You could add this to Tests.ocf\OverlayTest.ocs, if it is not already in there. I think it makes sense to add it. |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2016-04-17 21:05 | Maikel | New Issue | |
2016-04-17 21:05 | Maikel | Status | new => assigned |
2016-04-17 21:05 | Maikel | Assigned To | => Sven2 |
2016-04-17 21:08 | Maikel | Assigned To | Sven2 => |
2016-04-17 21:08 | Maikel | Note Added: 0005061 | |
2016-04-17 21:08 | Maikel | Note Added: 0005062 | |
2016-04-18 06:15 | Maikel | Assigned To | => Maikel |
2016-04-18 06:15 | Maikel | Status | assigned => new |
2016-04-18 15:46 | Sven2 | Note Added: 0005065 | |
2016-04-18 15:46 | Sven2 | Note Edited: 0005065 | View Revisions |
2016-04-18 18:04 | Maikel | Note Added: 0005066 | |
2016-04-20 11:06 | Maikel | Assigned To | Maikel => |
2016-04-21 19:21 | Marky | Note Added: 0005071 |