Skip to content

[v6] migrate most CCLabelBMFont usage to geode::Label - #2185

Open
Fleeym wants to merge 1 commit into
v6from
feat/v6-label-migration
Open

[v6] migrate most CCLabelBMFont usage to geode::Label#2185
Fleeym wants to merge 1 commit into
v6from
feat/v6-label-migration

Conversation

@Fleeym

@Fleeym Fleeym commented Sep 10, 2026

Copy link
Copy Markdown
Member

Tried to not just find & replace, I attempted to optimize create() and setString() usage wherever I could.

This obviously will lead to some API breaks, so I'm basing it on v6.

I think this is the full list:

  • SettingNodeV3::getNameLabel(): changed return type
  • SettingNodeV3::getStatusLabel(): changed return type
  • IconButtonSprite::getLabel(): changed return type
  • Notification::getLabel(): changed return type
  • Popup::m_title: is now geode::Label
  • ProgressBar::getProgressLabel(): changed return type
  • SelectList::m_label: is now geode::Label
  • SliderNode::linkLabel(): requires geode::Label for 1st param
  • SliderNode::getLinkedLabel(): changed return type
  • SimpleTextArea::getLines(): returns vector<Label*>

The majority of those should not be issues for most devs, besides Popup, SliderNode and SelectList.

Comment thread loader/src/ui/nodes/IconButtonSprite.cpp Outdated
Comment thread loader/src/ui/mods/settings/SettingNodeV3.cpp Outdated
Tried to not just find & replace, I attempted to optimize create()
and setString() usage wherever I could.

This obviously will lead to some API breaks, so I'm basing it on v6.

I think this is the full list:

- `SettingNodeV3::getNameLabel()`: changed return type
- `SettingNodeV3::getStatusLabel()`: changed return type
- `IconButtonSprite::getLabel()`: changed return type
- `Notification::getLabel()`: changed return type
- `Popup::m_title`: is now Label
- `ProgressBar::getProgressLabel()`: changed return type
- `SelectList::m_label`: is now Label
- `SliderNode::linkLabel()`: requires geode::Label for 1st param
- `SliderNode::getLinkedLabel()`: changed return type
- `SimpleTextArea::getLines()`: returns vector<Label*>

Most should not be issues for most devs, besides Popup, SliderNode
and SelectList.
@Fleeym
Fleeym force-pushed the feat/v6-label-migration branch from f6f69c3 to be1e5a7 Compare September 10, 2026 20:07
@Fleeym

Fleeym commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

Also updated TextInput to call m_impl->label->setLimitLabelWidth() when TextInput::setWidth() is called.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants