Add a tooltip showing the price of a room tile/trap when hovering its icon

[akien-mga]

As the topic says. For now it's by trial and error that you know whether you can place a room tile.

[hwoarangmy]

Yes, we could add in the text showing the room name the total price of the room.

[akien-mga]

Yes, we could add in the text showing the room name the total price of the room.

Ah indeed it could be done like that too. Or both :-)

[Bertram25]

IMHO, both.

When selecting the room, its price should be indicated next to the name.
When in the process of adding a room (mouse dragging), the total price should be shown.
Later, we'll probably want to color the stuff depending on whether it's possible.

[Bertram25]

0.4.9 or later? (Might be trickier than it sounds... or not.)

[hwoarangmy]

doesn't sound that tricky. If it reveals to be, we could just forget for now :)

[Bertram25]

Doc for default CEGUI tooltip implementation:
http://cegui.org.uk/wiki/Tooltips
http://static.cegui.org.uk/docs/0.8.4/classCEGUI_1_1Tooltip.html

I'll deal with improving the skin once we have it in-game.

[Bertram25]

As discussed with @hwoarangmy , the CEGUI button icon tooltip part is moved to 'Later' for now.
As he said, all the necessary code is here to show cegui system tooltips, yet they won't show for an unknown reason. This will have to be investigated.
And if we can't display them that way, we will have to add a custom text overlay and use it as tooltip.

[Bertram25]

Reopening as the hover tooltip part isn't done yet.

[Bertram25]

Note to self or anyone, for later use when fixing this:
I've looked at that by pure fanciness:
First problem I have seen is that in Gui.cpp, we're using:

CEGUI::System::getSingleton().getDefaultGUIContext().setDefaultTooltipObject(new CEGUI::Tooltip("OD","Tooltip"));

Which creates a memleak and isn't the preferred way to do that. We should instead use AFAIK:

CEGUI::System::getSingleton().getDefaultGUIContext().setDefaultTooltipType((CEGUI::utf8*)"OD/Tooltip");

As it will handle the tooltip internally, and log if there is an error when creating a valid tooltip object.

Secondly, in ODFrameListener, the injectTimePulse() method should also be called on the default gui context. This line is missing next to the existing one:

CEGUI::System::getSingleton().getDefaultGUIContext().injectTimePulse(evt.timeSinceLastFrame);

This may or may not fix the problem but at least can't hurt.