gtk-rs / gtk

DEPRECATED, use https://github.com/gtk-rs/gtk3-rs repository instead!

Home Page:https://gtk-rs.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

LabelBuilder::wrap_mode could use gtk::WrapMode, not pango::WrapMode

Dieff opened this issue · comments

The wrap_mode method on LabelBuilder takes apango::wrap_mode struct as an argument. However, the gtk library also exports an enum called WrapMode which appears to be almost the same (just with the addition of a None variant). Every place within gtk functions that I can find uses the pango version of WrapMode.

It seems to me that the gtk::WrapMode is redundant, and maybe if it were changed than all of functions which use pango::WrapMode could use gtk::WrapMode instead. This would simplify the use of the library, because it wouldn't be necessary to import pango just to set the wrapping mode of gtk widgets.

Except the wrapping is handled by pango. The gtk::WrapMode is only used in GtkTextView I guess. Anyway, there's nothing that can be done here. Adding a Into<gtk::WrapMode> for pango::WrapMode doesn't make sense.