Weird status icons stacking - empty placeholders #428

issue katesalazar openend this issue on September 18, 2021
  1. katesalazar commented at 5:30 pm on September 18, 2021: contributor

    There are at least up to six icons in order:

    1. unit: btc, sat…
    2. HD: disabled/enabled.
    3. wallet lock.
    4. proxy (disabled/enabled ?).
    5. network status and number of connections.
    6. syncing/synced.

    I think there’s a way to make a ghostly placeholder appear between the unit (btc, sat…) and the proxy “P”.

    Expected behavior

    A normal stacking of icons.

    Actual behavior

    Too much spacing between the unit and the proxy “P”.

    To reproduce

    Boot Bitcoin Core Qt without a wallet loaded. Just happens, in my case…

    System information

    Self-compiled v22.0 tag.

    Contemporary Debian GNU/Linux, KDE 5.54.0, Qt 5.11.3, xcb.

    Other

    I might have some time to bisect this.

  2. katesalazar added the label Bug on Sep 18, 2021
  3. katesalazar commented at 7:39 pm on September 18, 2021: contributor
    the v0.21.0 tag showing the same as the v22.0 tag
  4. katesalazar commented at 7:55 pm on September 18, 2021: contributor
    In the v0.20.0 I’m having a hard time to have it not autoloading or autocreating a wallet on startup, so no more bisecting for now.
  5. hebasto commented at 12:33 pm on September 29, 2021: member

    @katesalazar

    Screenshots could be very helpful to describe a visual issue :)

    FWIW, the relevant code is here: https://github.com/bitcoin-core/gui/blob/d648bbb0a7909aed953b6f7907690134606a033a/src/qt/bitcoingui.cpp#L159-L181

  6. hebasto removed the label Bug on Sep 29, 2021
  7. hebasto added the label Design on Sep 29, 2021
  8. katesalazar commented at 6:02 pm on September 29, 2021: contributor

    Alright.






    I see an unnecessary placeholder in figures 1, 2, 3; filled in figure 4.

    I would have a look at the code. I wonder if this is one of those QT snake pit issues, like that one about not showing capslock active until an alphabetic character is pressed. It’s suspicious this is not dealed with.

  9. hebasto commented at 7:08 pm on September 29, 2021: member

    @katesalazar

    Fixed in #439.

  10. hebasto removed the label Design on Sep 29, 2021
  11. hebasto added the label UI on Sep 29, 2021
  12. katesalazar commented at 7:20 pm on September 29, 2021: contributor
    Oh, sweet. Will watch it right away. Thanks. ❤️
  13. katesalazar commented at 8:49 pm on September 29, 2021: contributor

    FWIW, the relevant code is here

    That was inviting, and I suppose I should have been able to figure this one out, but I’m swamped real deep with another PR and specially a couple tons of non-work life stuff. 😭

    I think that in around 5 to 7 years I’ll be more free to tinker in the bitcoin landscape.

    Meanwhile, I hope I keep reporting GUI sightings (and test the QML fork), cheers.

  14. jarolrod commented at 1:18 am on September 30, 2021: member
    I was able to replicate the bug when starting with no wallet
  15. hebasto closed this on Sep 30, 2021

  16. katesalazar renamed this:
    Weird status icons stacking - ghost icon
    Weird status icons stacking - empty placeholders
    on Oct 6, 2021
  17. katesalazar referenced this in commit 1e56bd5993 on Oct 14, 2021
  18. bitcoin-core locked this on Oct 6, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/gui. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-23 00:20 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me