[haiku-bugs] Re: [Haiku] #16488: Icon-O-Matic Properties Input Field entries out of scope

  • From: "Haiku" <trac@xxxxxxxxxxxx>
  • To: undisclosed-recipients: ;
  • Date: Mon, 17 Aug 2020 19:40:37 -0000

#16488: Icon-O-Matic Properties Input Field entries out of scope
---------------------------------+--------------------------
  Reporter:  roiredxsoto         |      Owner:  jscipione
      Type:  bug                 |     Status:  assigned
  Priority:  normal              |  Milestone:  Unscheduled
 Component:  Kits/Interface Kit  |    Version:  R1/beta2
Resolution:                      |   Keywords:  Icon-O-Matic
Blocked By:                      |   Blocking:
  Platform:  All                 |
---------------------------------+--------------------------
Comment (by jscipione):

 I can confirm that hrev54496 is the culprit here. I have updated my
 revised patchset to fix this bug by going with the latter solution,
 removing the offending code and going with the default text view behavior.
 Calling SetInsets(0, 0, 0, 0); also fixes the problem.

 The general problem here is code that looks like this:
 {{{
 void
 MyTextView::FrameResized(float width, float height)
 {
     BRect textRect(TextRect());
     textRect.right = textRect.left + width;
     SetTextRect(textRect);
 }
 }}}

 This code falls down because the default insets get applied again and
 again as SetTextRect() is called shrinking the text rect until it can’t be
 seen anymore. So if this pattern is used elsewhere in apps we can’t update
 then we may not be able to make them work unless we set the default
 textview insets to 0.
-- 
Ticket URL: <https://dev.haiku-os.org/ticket/16488#comment:4>
Haiku <https://dev.haiku-os.org>
The Haiku operating system.

Other related posts: