bitcoin-core / gui-qml

Bitcoin GUI (experimental QML-based fork)
MIT License
111 stars 40 forks source link

Icon control #355

Closed promag closed 1 year ago

promag commented 1 year ago

Button control is used to show icons because it has support to colorize the image. Button is packed with more stuff like states, event handling, label... eventually a light component should be used instead. For now, abstract the usage of Button in the new Icon control and use it in trivial places. There are more places where Icon can be used but require more changes, so I'm leaving them for follow-ups.

promag commented 1 year ago

Forgot to mention that this is a small step in fixing focus navigation since the previous Button instances had by default enabled: true and so were in the focus chain.