Results 95 comments of jarolrod

Updated from [4909ba5 ](https://github.com/bitcoin-core/gui-qml/commit/4909ba591834840cb0238c95a1ceeb8beecb0942)to [02d3121](https://github.com/bitcoin-core/gui-qml/commit/02d3121218e1dc04582a85149cc3eabe795aef03) Addressed review comments on id naming and icon size, additionally rebased over main to address conflicts. TODO's to be left for follow-ups

Leaving this open as while we don't have the original issue, we still are not [adherent](https://github.com/bitcoin-core/gui-qml/issues/393#issuecomment-2045264340) to the block clock design (in several ways)

https://www.qt.io/download-open-source#:~:text=Open%20Source%20Usage%20Obligations&text=When%20developing%20under%20this%20license,code%20copy%20available%20for%20customers

The acknowledgment text could be the same as the one used within the Qt Widgets gui, pictured below. This text and window comes from the [`QApplication::aboutQt`](https://doc.qt.io/qt-6/qapplication.html#aboutQt) function, which we don't...

Yes this is planned, first we want to make sure tab navigation is focused properly which at least depends on the move to stack views, soon TM

Filled, Hover, and Active being added in https://github.com/bitcoin-core/gui-qml/pull/241

Confirming this is still an issue, need to read from qsettings what is enabled, i think this was skipped over because previously we were setting an integer value for what...

https://github.com/bitcoin-core/gui-qml/pull/243 introduces disabled https://github.com/bitcoin-core/gui-qml/pull/256 introduces EMPTY state The remaining states are - Error

concept ack, only thing is we wouldn't be able to change the network from the gui, outside of that, I like this more than a colored icon