Release Cleanup: Day 5
Today I focused on the
Widgets/header package, but as that’s mostly code sourced from other elementary projects there wasn’t much to clean up leaving me to focus on the
AddressBar was already very nicely written, in part because of how simple it is, but I did manage to find some things to correct.
At the lowest end there was a tiny bit of defunct code and expressions which could be written more concisely. At the other end it didn’t quite take up all available space at certain window sizes and clicking an autocompletion didn’t navigate to that link.
The thing with sizing the addressbar is that left to it’s own devices a text entry allocates itself a “natural size” that’s much smaller than what I need. At the same time the headerbar doesn’t know to expand it’s title widgets to fit all available space, so those are left at their “natural size”1. To fix this the natural size has to be much bigger so the headerbar knows to fill all available space with it.
As for the fact you couldn’t click an autocompletion to navigate to it’s page, that was a surprisingly quick fix. I previously called two methods in the wrong order, causing an event handler to be added to the completions list before it existed. That’s no longer an issue.
After that I attempted to upgrade the ButtonWithMenu code I’m using to be inline with Pantheon Files2 and tidied up the widgets which overlay ontop of webpages. While there were one or two codestyle issues to fix, most the problems were bigger than that but yet had trivial solutions.
FindToolbar had two issues: a bit of menu positioning logic was extra clutter and it was using deprecated APIs to turn the text red when some text can’t be found on the page. The former was fixed by utilizing the
ButtonWithMenu class I was using elsewhere. And the latter was fixed by switching to a newer API which is more concise and semantic, basically it involves applying a
.error class to the entry just as one would on the Web.
insensitive) while it’s
InfoBar is open. On it’s own that doesn’t communicate that the page is inactive, but that can be solved simply by utilizing the statusbar text.
- Or at least that’s the behaviour I found while implementing the addressbar. Things might have changed now.
- Turns out there weren’t any changes to this file to be commited.