Skip to content

Commit

Permalink
Fix invalid minimum size for translated messages in option button
Browse files Browse the repository at this point in the history
  • Loading branch information
Dragoncraft89 committed Jun 29, 2023
1 parent 16dd4e5 commit c33748d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion scene/gui/option_button.cpp
Expand Up @@ -451,7 +451,7 @@ void OptionButton::_refresh_size_cache() {

_cached_size = Vector2();
for (int i = 0; i < get_item_count(); i++) {
_cached_size = _cached_size.max(get_minimum_size_for_text_and_icon(get_item_text(i), get_item_icon(i)));
_cached_size = _cached_size.max(get_minimum_size_for_text_and_icon(popup->get_item_xl_text(i), get_item_icon(i)));
}
update_minimum_size();
}
Expand Down
5 changes: 5 additions & 0 deletions scene/gui/popup_menu.cpp
Expand Up @@ -1471,6 +1471,11 @@ String PopupMenu::get_item_text(int p_idx) const {
return items[p_idx].text;
}

String PopupMenu::get_item_xl_text(int p_idx) const {
ERR_FAIL_INDEX_V(p_idx, items.size(), "");
return items[p_idx].xl_text;
}

Control::TextDirection PopupMenu::get_item_text_direction(int p_idx) const {
ERR_FAIL_INDEX_V(p_idx, items.size(), Control::TEXT_DIRECTION_INHERITED);
return items[p_idx].text_direction;
Expand Down
1 change: 1 addition & 0 deletions scene/gui/popup_menu.h
Expand Up @@ -250,6 +250,7 @@ class PopupMenu : public Popup {
void toggle_item_checked(int p_idx);

String get_item_text(int p_idx) const;
String get_item_xl_text(int p_idx) const;
Control::TextDirection get_item_text_direction(int p_idx) const;
String get_item_language(int p_idx) const;
int get_item_idx_from_text(const String &text) const;
Expand Down

0 comments on commit c33748d

Please sign in to comment.