Hi Albrecht! On 01/22/2020 04:04:09 PM Wed, Albrecht Dreß wrote:
Hi all, I currently try to convince a colleague to use Balsa, and he mentioned some issues (more to come later) – here is a patch for a few more cosmetic ones which are easy to fix: - If the signature status is /very/ long (as for an expired key in German translation), the message in the header box or in the expander label at the bottom may prevent the message from shrinking horizontally. Solution: enable label line wrap - In the signature status box at the bottom, if the labels in the 2nd column wrap, the label in the 1st column is vertically centred which looks ugly. Solution: use gtk_label_set_yalign() - The wrapped label is centred in the column, not aligned at the left. Solution: use gtk_label_set_xalign() - The meaning of the blue/green/yellow/red padlock icons is not obvious. Solution: add tooltip (this may be controversial as we didn't use tooltips yet, but it's an easy and intuitive solution). The attached patch is suitable for both the master and gmime branches. Note that it may produce offset warnings in combination with the “Bug fix: S/MIME cert chain broken in identity key selection” I sent earlier this evening, but should work anyway. BTW, the gtk_label_set_xalign(…, 0.0F) trick might be helpful in several other places, too – e.g. for the message headers. All long ones (like Received:) look really ugly! Opinions? Cheers, Albrecht. --- Patch details: - libbalsa/libbalsa-gpgme-widgets.c: improve xalign and yalign properties of labels in signature and key details widgets - src/balsa-mime-widget-crypto.c: balsa_mime_widget_signature_widget: wrap/align signature status in expander label; balsa_mime_widget_crypto_frame: add tooltips to padlock icons - src/balsa-mime-widget-message.c: add_header_sigstate: wrap/align signature status in headers box
Thanks for the patch--looks good! For myself, I don't see the tooltip as controversial! I just pushed both of your patches to GitLab, and another setting the xalign on those ugly headers--thanks for the suggestion! Best, Peter
Attachment:
pgpn3zuqlfcTF.pgp
Description: PGP signature