2018-08-10 18:09:47 +02:00
|
|
|
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
|
|
|
|
/*
|
|
|
|
* This file is part of the LibreOffice project.
|
|
|
|
*
|
|
|
|
* This Source Code Form is subject to the terms of the Mozilla Public
|
|
|
|
* License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
|
|
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
|
|
|
|
*
|
|
|
|
* This file incorporates work covered by the following license notice:
|
|
|
|
*
|
|
|
|
* Licensed to the Apache Software Foundation (ASF) under one or more
|
|
|
|
* contributor license agreements. See the NOTICE file distributed
|
|
|
|
* with this work for additional information regarding copyright
|
|
|
|
* ownership. The ASF licenses this file to you under the Apache
|
|
|
|
* License, Version 2.0 (the "License"); you may not use this file
|
|
|
|
* except in compliance with the License. You may obtain a copy of
|
|
|
|
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
|
|
|
|
*/
|
|
|
|
|
2021-09-17 14:54:33 +10:00
|
|
|
#pragma once
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2023-07-19 08:50:50 +03:00
|
|
|
#include <basegfx/point/b2dpoint.hxx>
|
2018-08-10 18:09:47 +02:00
|
|
|
#include <basegfx/polygon/b2dpolypolygon.hxx>
|
2024-04-10 12:15:55 +05:00
|
|
|
#include <basegfx/range/b2drectangle.hxx>
|
tdf#30731: Improve caret travelling in Writer
Previously, when measuring caret position, Writer would measure the
width of the substring before the caret (i.e. layout it independent of
the text after the caret and measure its width).
This is incorrect, though. It assumes cutting the string laying it out
would result in the same width as when laid out as part of a bigger
string, which is invalid assumption when e.g. cutting inside a ligature
or between letters that have different shapes when next to each other,
etc.
This appears to work when the width of the substring laid out alone is
close enough to its width when laid out with the full text. But in cases
where is widths are largely different, like the extreme case in the bug
report, the caret will be jumping around as it is positioned based on
the unligated glyphs not the ligated, rendered glyphs.
This change introduces a special mode of measuring text width for caret
positioning, that will layout the whole string that return the width of
the requested substring.
Fields and small caps text are trickier to handle, so old behaviour is
retained for them. Now one will probably notice but if they do, it can
be dealt with then.
This also tries to be conservative and keep other pleases using the
existing behaviour which might be desirable (e.g. when measuring text
width for line breaking, we want the unligated width), but there might
be other places that should use the new behaviour.
To handle caret inside ligatures, the grapheme clusters in the ligature
are counted and the width of the whole ligature is distributed on them
evenly. A further improvement would be using HarfBuzz API to get
ligature caret positions for fonts that provide them, which helps when
the ligature components have different widths.
Change-Id: I02062e2e2e1b1a35c8f84307c0a8f5d743059ab5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138889
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-08-26 22:20:55 +02:00
|
|
|
#include <i18nlangtag/languagetag.hxx>
|
2018-08-10 18:09:47 +02:00
|
|
|
#include <tools/gen.hxx>
|
2020-10-20 07:27:44 +02:00
|
|
|
#include <tools/degree.hxx>
|
2021-09-12 14:41:19 +10:00
|
|
|
|
2021-03-16 15:04:08 +01:00
|
|
|
#include <vcl/glyphitem.hxx>
|
2018-08-10 18:09:47 +02:00
|
|
|
#include <vcl/dllapi.h>
|
|
|
|
|
2021-09-03 14:44:22 +02:00
|
|
|
class LogicalFontInstance;
|
2021-03-07 13:59:04 +11:00
|
|
|
namespace vcl::text { class ImplLayoutArgs; }
|
2024-08-02 15:56:52 +02:00
|
|
|
namespace basegfx { class BColor; }
|
2018-08-10 18:09:47 +02:00
|
|
|
class SalGraphics;
|
2019-07-05 22:12:39 +02:00
|
|
|
class GlyphItem;
|
2018-12-26 01:11:42 +01:00
|
|
|
|
2018-08-10 18:09:47 +02:00
|
|
|
// all positions/widths are in font units
|
|
|
|
// one exception: drawposition is in pixel units
|
|
|
|
|
|
|
|
// Unfortunately there is little documentation to help implementors of
|
|
|
|
// new classes derived from SalLayout ("layout engines"), and the code
|
|
|
|
// and data structures are far from obvious.
|
|
|
|
|
|
|
|
// For instance, I *think* the important virtual functions in the
|
|
|
|
// layout engines are called in this order:
|
|
|
|
|
|
|
|
// * LayoutText()
|
|
|
|
// * AdjustLayout(), any number of times (but presumably
|
|
|
|
// usually not at all or just once)
|
|
|
|
// * Optionally, DrawText()
|
|
|
|
|
|
|
|
// Functions that just return information like GetTexWidth() and
|
|
|
|
// FillDXArray() are called after LayoutText() and before DrawText().
|
|
|
|
|
|
|
|
// Another important questions is which parts of an ImplLayoutArgs can
|
|
|
|
// be changed by callers between LayoutText() and AdjustLayout()
|
|
|
|
// calls. It probably makes sense only if one assumes that the "string
|
|
|
|
// related inputs" part are not changed after LayoutText().
|
|
|
|
|
|
|
|
// But why use the same ImplLayoutArgs structure as parameter for both
|
|
|
|
// LayoutText() and AdjustLayout() in the first place? And why
|
|
|
|
// duplicate some of the fields in both SalLayout and ImplLayoutArgs
|
|
|
|
// (mnMinCharPos, mnEndCharPos, mnLayoutFlags==mnFlags,
|
|
|
|
// mnOrientation)? Lost in history...
|
|
|
|
|
|
|
|
class VCL_DLLPUBLIC SalLayout
|
|
|
|
{
|
|
|
|
public:
|
|
|
|
virtual ~SalLayout();
|
|
|
|
// used by upper layers
|
2023-07-19 08:50:50 +03:00
|
|
|
basegfx::B2DPoint& DrawBase() { return maDrawBase; }
|
|
|
|
const basegfx::B2DPoint& DrawBase() const { return maDrawBase; }
|
2024-08-21 16:54:44 -06:00
|
|
|
basegfx::B2DPoint& DrawOffset() { return maDrawOffset; }
|
|
|
|
const basegfx::B2DPoint& DrawOffset() const { return maDrawOffset; }
|
2023-07-19 08:50:50 +03:00
|
|
|
basegfx::B2DPoint GetDrawPosition( const basegfx::B2DPoint& rRelative = basegfx::B2DPoint(0,0) ) const;
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2021-03-07 13:59:04 +11:00
|
|
|
virtual bool LayoutText( vcl::text::ImplLayoutArgs&, const SalLayoutGlyphsImpl* ) = 0; // first step of layouting
|
|
|
|
virtual void AdjustLayout( vcl::text::ImplLayoutArgs& ); // adjusting after fallback etc.
|
2018-08-10 18:09:47 +02:00
|
|
|
virtual void DrawText( SalGraphics& ) const = 0;
|
|
|
|
|
2020-10-20 07:27:44 +02:00
|
|
|
Degree10 GetOrientation() const { return mnOrientation; }
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2023-07-17 12:27:03 +03:00
|
|
|
void SetSubpixelPositioning(bool bSubpixelPositioning)
|
2022-01-16 17:24:08 +00:00
|
|
|
{
|
2023-07-17 12:27:03 +03:00
|
|
|
mbSubpixelPositioning = bSubpixelPositioning;
|
2022-01-16 17:24:08 +00:00
|
|
|
}
|
|
|
|
|
2023-07-17 12:27:03 +03:00
|
|
|
bool GetSubpixelPositioning() const
|
2022-08-15 17:24:28 +01:00
|
|
|
{
|
2023-07-17 12:27:03 +03:00
|
|
|
return mbSubpixelPositioning;
|
2022-08-15 17:24:28 +01:00
|
|
|
}
|
|
|
|
|
2018-08-10 18:09:47 +02:00
|
|
|
// methods using string indexing
|
2023-07-16 16:06:59 +03:00
|
|
|
virtual sal_Int32 GetTextBreak(double nMaxWidth, double nCharExtra, int nFactor) const = 0;
|
|
|
|
virtual double FillDXArray( std::vector<double>* pDXArray, const OUString& rStr ) const = 0;
|
|
|
|
virtual double GetTextWidth() const { return FillDXArray( nullptr, {} ); }
|
2024-04-29 04:30:43 -06:00
|
|
|
|
|
|
|
virtual double FillPartialDXArray(std::vector<double>* pDXArray, const OUString& rStr,
|
|
|
|
sal_Int32 skipStart, sal_Int32 amt) const
|
|
|
|
= 0;
|
|
|
|
|
|
|
|
virtual double GetPartialTextWidth(sal_Int32 skipStart, sal_Int32 amt) const
|
|
|
|
{
|
|
|
|
return FillPartialDXArray(nullptr, {}, skipStart, amt);
|
|
|
|
}
|
|
|
|
|
2023-07-19 19:07:34 +03:00
|
|
|
virtual void GetCaretPositions( std::vector<double>& rCaretPositions, const OUString& rStr ) const = 0;
|
2024-10-18 06:12:39 -06:00
|
|
|
|
|
|
|
virtual bool HasFontKashidaPositions() const = 0;
|
2022-08-12 02:12:04 +02:00
|
|
|
virtual bool IsKashidaPosValid ( int /*nCharPos*/, int /*nNextCharPos*/ ) const = 0; // i60594
|
2018-08-10 18:09:47 +02:00
|
|
|
|
|
|
|
// methods using glyph indexing
|
2023-07-19 08:50:50 +03:00
|
|
|
virtual bool GetNextGlyph(const GlyphItem** pGlyph, basegfx::B2DPoint& rPos, int& nStart,
|
2022-09-30 09:55:52 +02:00
|
|
|
const LogicalFontInstance** ppGlyphFont = nullptr) const = 0;
|
2018-10-28 13:41:10 +01:00
|
|
|
virtual bool GetOutline(basegfx::B2DPolyPolygonVector&) const;
|
2024-04-10 12:15:55 +05:00
|
|
|
bool GetBoundRect(basegfx::B2DRectangle&) const;
|
|
|
|
|
2025-03-19 21:28:33 +02:00
|
|
|
static tools::Rectangle BoundRect2Rectangle(const basegfx::B2DRectangle&);
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2021-03-16 15:04:08 +01:00
|
|
|
virtual SalLayoutGlyphs GetGlyphs() const;
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2024-08-02 15:56:52 +02:00
|
|
|
virtual void drawSalLayout(void* /*pSurface*/, const basegfx::BColor& /*rTextColor*/, bool /*bAntiAliased*/) const {}
|
|
|
|
|
2018-08-10 18:09:47 +02:00
|
|
|
protected:
|
|
|
|
// used by layout engines
|
2018-09-30 14:00:54 +00:00
|
|
|
SalLayout();
|
2018-08-10 18:09:47 +02:00
|
|
|
|
|
|
|
private:
|
2018-09-30 14:00:54 +00:00
|
|
|
SalLayout(const SalLayout&) = delete;
|
|
|
|
SalLayout& operator=(const SalLayout&) = delete;
|
2018-08-10 18:09:47 +02:00
|
|
|
|
|
|
|
protected:
|
|
|
|
int mnMinCharPos;
|
|
|
|
int mnEndCharPos;
|
tdf#30731: Improve caret travelling in Writer
Previously, when measuring caret position, Writer would measure the
width of the substring before the caret (i.e. layout it independent of
the text after the caret and measure its width).
This is incorrect, though. It assumes cutting the string laying it out
would result in the same width as when laid out as part of a bigger
string, which is invalid assumption when e.g. cutting inside a ligature
or between letters that have different shapes when next to each other,
etc.
This appears to work when the width of the substring laid out alone is
close enough to its width when laid out with the full text. But in cases
where is widths are largely different, like the extreme case in the bug
report, the caret will be jumping around as it is positioned based on
the unligated glyphs not the ligated, rendered glyphs.
This change introduces a special mode of measuring text width for caret
positioning, that will layout the whole string that return the width of
the requested substring.
Fields and small caps text are trickier to handle, so old behaviour is
retained for them. Now one will probably notice but if they do, it can
be dealt with then.
This also tries to be conservative and keep other pleases using the
existing behaviour which might be desirable (e.g. when measuring text
width for line breaking, we want the unligated width), but there might
be other places that should use the new behaviour.
To handle caret inside ligatures, the grapheme clusters in the ligature
are counted and the width of the whole ligature is distributed on them
evenly. A further improvement would be using HarfBuzz API to get
ligature caret positions for fonts that provide them, which helps when
the ligature components have different widths.
Change-Id: I02062e2e2e1b1a35c8f84307c0a8f5d743059ab5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138889
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-08-26 22:20:55 +02:00
|
|
|
LanguageTag maLanguageTag;
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2020-10-20 07:27:44 +02:00
|
|
|
Degree10 mnOrientation;
|
2018-08-10 18:09:47 +02:00
|
|
|
|
2024-08-21 16:54:44 -06:00
|
|
|
basegfx::B2DPoint maDrawOffset;
|
2023-07-19 08:50:50 +03:00
|
|
|
basegfx::B2DPoint maDrawBase;
|
2022-01-16 17:24:08 +00:00
|
|
|
|
2023-07-17 12:27:03 +03:00
|
|
|
bool mbSubpixelPositioning;
|
2018-08-10 18:09:47 +02:00
|
|
|
};
|
|
|
|
|
|
|
|
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
|