[haiku-commits] haiku: hrev52942 - src/kits/tracker

  • From: Adrien Destugues <pulkomandy@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 28 Feb 2019 14:10:27 -0500 (EST)

hrev52942 adds 1 changeset to branch 'master'
old head: 1f0635d2277dcd0818dc7f539c1cb1b296f6444b
new head: 579b9cd9b9d49263bd72c49faa3504d0b2709466
overview: 
https://git.haiku-os.org/haiku/log/?qt=range&q=579b9cd9b9d4+%5E1f0635d2277d

----------------------------------------------------------------------------

579b9cd9b9d4: Adjust the magic incantations to fix spacing
  
  This is my first commit, so bear with me if I've violated any standards here!
  
  I've bumped a few offsets to fix text clipping in the Get Info window. The 
proper
  long-term fix is to recreate this window with the layout library, but that's
  a substantially larger job.
  
  Patch set 1
  Before: https://i.imgur.com/S7Pl5Qv.png
  After: https://i.imgur.com/bd3H1Kw.png
  
  Patch set 3
  French: https://i.imgur.com/rpmUb5T.png
  German: https://i.imgur.com/ca9DecW.png
  Portuguese: https://i.imgur.com/dE8sKFI.png
  
  The font size in the Permissions drop-down is fixed. I had previously bumped 
it to
  12, to be inline with the default font size present in a new Haiku install. 
However,
  that produced text clipping for French and other locales. I reverted it back 
to 10,
  and now longer strings fit as-is.
  
  Change-Id: I7f4412b10074c76eb5b023a231bdb6b230c8f35a
  Reviewed-on: https://review.haiku-os.org/c/1073
  Reviewed-by: waddlesplash <waddlesplash@xxxxxxxxx>

                                [ Zach Dykstra <dykstra.zachary@xxxxxxxxx> ]

----------------------------------------------------------------------------

Revision:    hrev52942
Commit:      579b9cd9b9d49263bd72c49faa3504d0b2709466
URL:         https://git.haiku-os.org/haiku/commit/?id=579b9cd9b9d4
Author:      Zach Dykstra <dykstra.zachary@xxxxxxxxx>
Date:        Thu Feb 21 05:48:51 2019 UTC
Committer:   Adrien Destugues <pulkomandy@xxxxxxxxx>
Commit-Date: Thu Feb 28 19:10:12 2019 UTC

----------------------------------------------------------------------------

1 file changed, 6 insertions(+), 6 deletions(-)
src/kits/tracker/FilePermissionsView.cpp | 12 ++++++------

----------------------------------------------------------------------------

diff --git a/src/kits/tracker/FilePermissionsView.cpp 
b/src/kits/tracker/FilePermissionsView.cpp
index c0028f56e7..baf2b007c6 100644
--- a/src/kits/tracker/FilePermissionsView.cpp
+++ b/src/kits/tracker/FilePermissionsView.cpp
@@ -63,7 +63,7 @@ public:
        void Draw(BRect invalidate)
        {
                RotateBy(-M_PI / 5);
-               TranslateBy(0, Bounds().Height() / 1.5);
+               TranslateBy(0, Bounds().Height() / 3);
                BStringView::Draw(invalidate);
        }
 };
@@ -182,7 +182,7 @@ FilePermissionsView::FilePermissionsView(BRect rect, Model* 
model)
        }
 
        const float kTextControlLeft = 170, kTextControlRight = 270,
-               kTextControlTop = kRowLabelTop - 19,
+               kTextControlTop = kRowLabelTop - 29,
                kTextControlHeight = 14, kTextControlSpacing = 16;
 
        strView = new BStringView(BRect(kTextControlLeft, kTextControlTop,
@@ -202,9 +202,9 @@ FilePermissionsView::FilePermissionsView(BRect rect, Model* 
model)
        AddChild(fOwnerTextControl);
 
        strView = new BStringView(BRect(kTextControlLeft,
-                       kTextControlTop + 5 + 2 * kTextControlSpacing,
+                       kTextControlTop + 11 + 2 * kTextControlSpacing,
                        kTextControlRight,
-                       kTextControlTop + 2 + 2 * kTextControlSpacing
+                       kTextControlTop + 11 + 2 * kTextControlSpacing
                                + kTextControlHeight),
                "", B_TRANSLATE("Group"));
        strView->SetAlignment(B_ALIGN_CENTER);
@@ -212,9 +212,9 @@ FilePermissionsView::FilePermissionsView(BRect rect, Model* 
model)
        AddChild(strView);
 
        fGroupTextControl = new BTextControl(BRect(kTextControlLeft,
-                       kTextControlTop + 3 * kTextControlSpacing,
+                       kTextControlTop + 10 + 3 * kTextControlSpacing,
                        kTextControlRight,
-                       kTextControlTop + 3 * kTextControlSpacing + 
kTextControlHeight),
+                       kTextControlTop + 10 + 3 * kTextControlSpacing + 
kTextControlHeight),
                "", "", "", new BMessage(kNewGroupEntered));
        fGroupTextControl->SetDivider(0);
        AddChild(fGroupTextControl);


Other related posts:

  • » [haiku-commits] haiku: hrev52942 - src/kits/tracker - Adrien Destugues