TextArea cursor does not regard font size

Bug #1391335 reported by Michael Zanetti
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Medium
Cris Dywan
Vivid
New
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu RTM)
New
Undecided
Unassigned

Bug Description

Set this html text to a TextArea:

<!DOCTYPE html><html><head><meta name="viewport" content="width=device-width, initial-scale=1.0"/></head><body>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; padding-left:0px; text-indent:0px;">fsdf fsd <span style=" font-size:24pt;">fsd</span></p></body></html>

move the cursor between the different sized text elements.

Actual:
The cursor stays the same

Expected:
The cursor should scale with the text size

Related branches

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit:
assignee: nobody → Christian Dywan (kalikiana)
status: New → Confirmed
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit:
importance: Undecided → Medium
status: Confirmed → In Progress
Revision history for this message
Cris Dywan (kalikiana) wrote :

I pushed a branch for making the cursor change size depending on the cursor height. Note that apparently QML's TextInput always assumes the largest font size in any one line. So it seems this requires an additional upstream fix to work as requested.

Revision history for this message
Michael Zanetti (mzanetti) wrote :

You're right. However, using the upstream TextArea I did not notice this as a bug while the smaller cursor of Ubuntu's TextArea immediately jumped into my eyes. So I guess as long as the cursor is never smaller as the current font size we're good.

Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → Medium
status: New → In Progress
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.2.1450+15.04.20150318-0ubuntu1

---------------
ubuntu-ui-toolkit (1.2.1450+15.04.20150318-0ubuntu1) vivid; urgency=medium

  [ Timo Jyrinki ]
  * Clean QML cache after updating package, before reboot. LP: #1431228

  [ Zoltán Balogh ]
  * Improved test plan script. Fixes LP: #1430201

  [ Christian Dywan ]
  * New text handler visuals and improved behavior.
    Fixes LP: #1391335, LP: #1407723, LP: #1415548
  * Split off test verification and run it once only for 'make check'.
    Fixes LP: #1431698
  * Tweak assert_discard_popover to work better with textfield in header.
    Fixes LP: #1432611

  [ Tim Peeters ]
  * Make the filenames for header and navigation tests consistent and
    split up the big tabbed header.qml into multiple smaller test files.
    Fixes LP: #1429838

  [ Loïc Molinari ]
  * Optimized Slider and ProgressBar components. Switched to the native
    UbuntuShapeOverlay instead of PartialColorizeUbuntuShape to render
    the progression in the Slider and ProgressBar components. The
    rendering is now faster since it removes an FBO indirection and
    enables batching by the QtQtuick renderer.
    Fixes LP: #1398370, LP: #1427169
  * Added a dedicated overlay shape. Made use of the new UbuntuShape
    architecture to create an extended shape with efficient overlay
    rendering. Fixes LP: #1396170
  * Removed misleading statement from UbuntuShape documentation.
    Fixes LP: #1430198
  * Renamed the overlayGeometry property to overlayRect. The
    overlayGeometry being of type QRect, using overlayRect instead
    makes it more precise, less generic. Fixes LP: #1430278
  * Converted sourceOpacity from float to qreal. Respect the same
    guideline QtQuick items use to expose floating-point properties.
    That doesn't break the QML API since there's no precision loss
    from single to double precision, the cast is implicit.
    Fixes LP: #1430267
  * Fixed embedded QML code snippets indentation. Fixes LP: #1430274

  [ Florian Boucault ]
  * CheckBox: apply clipping to the Ubuntu shape and not to the CheckBox
    itself so that the tick never overflows. Fixes LP: #1430191

  [ Daniel Holbach ]
  * A number of links changed on design.ubuntu.com - let's make sure
    they're fixed in our docs. Fixes LP: #1428051

  [ CI Train Bot ]
  * New rebuild forced.
 -- CI Train Bot <email address hidden> Wed, 18 Mar 2015 12:23:35 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.