CEGUILayoutEditor Version 0.5 RC1 Build Notes....

[Van]

Could we please re-add the file mask *.* to the open layout dialog box? Not everyone uses *.layout as their file extension.

[EDIT] Has been added to Mantis: http://www.cegui.org.uk/mantis/view.php?id=66 [/EDIT]

[Rackle]

I would like the widgets to be sorted in ascending order; it would make it easier to locate a particular widget.

I would like the "Available widgets" pane/region/box to be larger such that more widgets are available; I spend too much time scrolling.

I would like the last used look (Vanilla, TaharezLook, WindowsLook) to be opened/expanded by default.

[EDIT] All three have been added to Mantis: http://www.cegui.org.uk/mantis/view.php?id=68 [/EDIT]

Jeez, I sound like a kid...

[Xirnohn]

I would like to see a property to edit tooltip texts.

[Xirnohn]

I would like to see a property to edit tooltip texts.

So, i added it myself.

What i changed:
-Added BackgroundEnabled property. Usefull for StaticText.
-Added Vertical/Horizontal Window alignment property, you can align it relaitve to the parent(i think you can use this from the Edit/Align menu. There are more options).
-Added Vertical/Horizontal Text alignment property. You can align your text in a StaticText.
-Added Image property. You can edit an StaticImage's Image property. I did it couse i hate when i save the layout and its delete the image property.
-Added InheritsTooltipText property.
-Added Tooltip property. Now you can edit window's tooltips.

I cant send patch from here, so i uploaded the modified source to here. Maybe somebody can make a patch.

[EDIT] I made a note so we don't forget to apply it: http://www.cegui.org.uk/mantis/view.php?id=69 [/EDIT]

Xir.

[scriptkid]

Hi all,

i have just come home from holidays, so i have to catch up for a bit

Xirnohn: thanks for the effort and the link, i will look at it!

I will put the other request in Mantis, including the ones mentioned in this thread: http://www.cegui.org.uk/phpBB2/viewtopic.php?t=1637

Thanks

[jfelser]

I would like to see hand added Window properties not included in CELayoutEditor at least be persistent through a load and resave. As it is, if I want to have certain properties I have to re-add them to the file by hand each time I save it in CELE.

Thanks alot, I appreciate the hard work you have put into this project, it has been very useful.

-JFelser

[Xirnohn]

What i changed:
-Added BackgroundEnabled property. Usefull for StaticText.
-Added Vertical/Horizontal Window alignment property, you can align it relaitve to the parent(i think you can use this from the Edit/Align menu. There are more options).
-Added Vertical/Horizontal Text alignment property. You can align your text in a StaticText.
-Added Image property. You can edit an StaticImage's Image property. I did it couse i hate when i save the layout and its delete the image property.
-Added InheritsTooltipText property.
-Added Tooltip property. Now you can edit window's tooltips.

If somebody cant compile it, i uploaded the compiled editor. Its for cegui 0.5.0.-RC1, dlls are included.


Bin & Source

[scriptkid]

Thanks Xirnohn,

i'll give it a try "soon"

[scriptkid]

Hi Xirnohn,

i'm applying your patch right now I have some questions regarding tooltips though. First, you have added a "fake" property for it, however the property "tooltip" is for tooltip text, IIRC. (At least that is what the documentation reads). The name might be confusing though, since there's also "InheritsTooltipText" and "TooltipType". So you might expect "TooltipText" or something...

Secondly, related to your fake property, i never see you calling the "setPropertyText" on a window. How do you apply changes to this property?

Thanks!

[Xirnohn]

Hi Xirnohn,

i'm applying your patch right now I have some questions regarding tooltips though. First, you have added a "fake" property for it, however the property "tooltip" is for tooltip text, IIRC. (At least that is what the documentation reads). The name might be confusing though, since there's also "InheritsTooltipText" and "TooltipType". So you might expect "TooltipText" or something...

Secondly, related to your fake property, i never see you calling the "setPropertyText" on a window. How do you apply changes to this property?

Thanks!

Hi.

Sorry for the late of reply.

Yes, a window's "Tooltip" property the tooltip text, but i think this spelling error regarding to CEGUI, not to me. The cell editor's first colum always the property's name, the secound the value. See the OnGridCellChange in the DialogMain.cpp.

Code: Select all

      wxString propertyName = m_grid->GetCellValue(row, COLUMN_NAME);
      wxString propertyValue = m_grid->GetCellValue(row, COLUMN_VALUE);
      m_document->SetProperty(StringHelper::ToCEGUIString(propertyName), StringHelper::ToCEGUIString(propertyValue));


If you change the "Tooltip" to "TooltipText" i think cegui raise an error when you change the property.

Secondly: I changed the "Tooltip"(and other properties i added) to "real" property(added TEX_TYPE and INT_TYPE, added them to supported properties), and its works fine. I did a mistake when i declared them fake.
Back to your question: i think every fake property cell editor invoke the OnGridCellChange, and in it the m_document->SetProperty(propertyName, propertyValue) sets the changed Tooltip text(becouse i dont changed the FAKE_PROPERTY_COUNT, so its go to the else state).

I hope i answered what you asked.

Xir.

[Xirnohn]

Well, i now work with "TooltipText", you work with it, then it saves "Tooltip".
But i think its still a CEGUI "error"!

Code: Select all

void DialogMain::CreateSupportedPropertyMap() add:
  m_supportedProperties.insert(make_pair("Tooltip", TEXT_TYPE));
  m_supportedProperties.insert(make_pair("TooltipText", TEXT_TYPE));

void DialogMain::AddRealProperties() change:

    if (*current == "Tooltip")
      AddPropertyRow("TooltipText", m_document->GetProperty (*current));
    else
      AddPropertyRow(*current, m_document->GetProperty (*current));

void DialogMain::OnGridCellChange(wxGridEvent& event) change
      wxString propertyName = m_grid->GetCellValue(row, COLUMN_NAME);
    if (propertyName == "TooltipText")
      propertyName = "Tooltip";
      wxString propertyValue = m_grid->GetCellValue(row, COLUMN_VALUE);
      m_document->SetProperty(StringHelper::ToCEGUIString(propertyName), StringHelper::ToCEGUIString(propertyValue));

wxGridCellEditor* DialogMain::GetPropertyEditor(const CEGUI::String& propertyName) add:
    case DialogMain::TEXT_TYPE:
      return new wxGridCellTextEditor();

And added PropertyType : TEXT_TYPE


Sorry, cant make diff/patch here...

Xir

[scriptkid]

Hi,

don't worry about the late reply. Your patch is in, and i am testing right now. I'll look at the tooltip myself again, i'm not sure what you mean by the difference(s) between Tooltip and Tooltip text. What i meant was just that Tooltip is the property we need. TooltipText does not exist, so it will indeed raise an exception when used...

[Xirnohn]

Hi,

don't worry about the late reply. Your patch is in, and i am testing right now. I'll look at the tooltip myself again, i'm not sure what you mean by the difference(s) between Tooltip and Tooltip text. What i meant was just that Tooltip is the property we need. TooltipText does not exist, so it will indeed raise an exception when used...

Hmm, now i feel myself idiotic, becouse then i dont understand you.
I wanted edit a window's tooltip text. In CEGUI its the Tooltip property. Adding fake properties was a little silly but easy thing.
I recognize that the name "Tooltip" a little confusing, and i changed the editor to display the "TooltipText" and change the "Tooltip" property.

I hope we get closer.

One another problem: when you try delete a text in the editor (Editbox's text, etc) the

Code: Select all

bool EditorDocument::SetProperty(const String& aName, const String& newValue)  line 819:
   if (newValue == "")
   {
      return true;
   }


lines return before the changes.

Xir