[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emacs-diffs] feature/extend_face_id 0a7b2b0 15/15: ; extend_face_id and
From: |
Jimmy Aguilar Mena |
Subject: |
[Emacs-diffs] feature/extend_face_id 0a7b2b0 15/15: ; extend_face_id and fill_column_indicator corrections |
Date: |
Wed, 9 Oct 2019 10:53:14 -0400 (EDT) |
branch: feature/extend_face_id
commit 0a7b2b0828c1f5afc073aea33fa2d139a84757da
Author: Jimmy Aguilar Mena <address@hidden>
Commit: Jimmy Aguilar Mena <address@hidden>
; extend_face_id and fill_column_indicator corrections
* Fix NEWS and add some comments in the code.
---
etc/NEWS | 7 ++++---
src/xdisp.c | 10 +++++++++-
src/xfaces.c | 2 +-
3 files changed, 14 insertions(+), 5 deletions(-)
diff --git a/etc/NEWS b/etc/NEWS
index 39994e1..9b039ba 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -385,10 +385,11 @@ simply, makes "foo" complete to both "barfoo" and
"frodo". Add 'flex'
to 'completion-styles' or 'completion-category-overrides' to use it.
+++
-** There is a new face attribute :extend to use the face attributes to
+** New face attribute ':extend' to control face extension at EOL.
+There is a new face attribute :extend to use the face attributes to
extend after the end of the line until the end of the window. Such
-:extend is set to nil by default in all faces except for `hl-line` and
-`region` because those extend until the end of the window by default.
+:extend is set to nil by default in all faces except for 'hl-line' and
+'region' because those extend until the end of the window by default.
** Connection-local variables
diff --git a/src/xdisp.c b/src/xdisp.c
index 1cd1b74..73a0b76 100644
--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -442,7 +442,7 @@ fill_column_indicator_column (struct it *it, int char_width)
? BVAR (current_buffer, fill_column)
: Vdisplay_fill_column_indicator_column);
- /* The stretch width needs to considet the latter
+ /* The stretch width needs to consider the latter
added glyph in append_space_for_newline. */
if (RANGED_FIXNUMP (0, col, INT_MAX))
{
@@ -21694,6 +21694,9 @@ extend_face_to_end_of_line (struct it *it)
&& indicator_column < it->last_visible_x)
{
+ /* Here we substract char_width because we want the
+ column indicator in the column INDICATOR_COLUMN, not
+ after it. */
const int stretch_width =
indicator_column - it->current_x - char_width;
@@ -21854,6 +21857,11 @@ extend_face_to_end_of_line (struct it *it)
default_face->id : face->id);
/* Display fill-column indicator if needed. */
+ /* We need to subtract 1 to the indicator_column here because we
+ will add the indicator IN the column indicator number, not
+ after it. We compare the variable it->current_x before
+ producing the glyph. When FRAME_WINDOW_P we substract
+ CHAR_WIDTH calculating STRETCH_WIDTH for the same reason. */
const int indicator_column =
fill_column_indicator_column (it, 1) - 1;
do
diff --git a/src/xfaces.c b/src/xfaces.c
index cc65527..5a741ae 100644
--- a/src/xfaces.c
+++ b/src/xfaces.c
@@ -2284,7 +2284,7 @@ filter_face_ref (Lisp_Object face_ref,
ATTR_FILTER is the index of a parameter that conditions the merging
for named faces (case 1) to only the face_ref where
- lface[merge_face_ref] is non-nil. To merge unconditionally set this
+ lface[merge_face_ref] is non-nil. To merge unconditionally set this
value to 0.
FACE_REF may be a single face specification or a list of such
- [Emacs-diffs] feature/extend_face_id 111dcf8 01/15: Simplify struct face to use only underline., (continued)
- [Emacs-diffs] feature/extend_face_id 111dcf8 01/15: Simplify struct face to use only underline., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id d8cb394 08/15: Make hl-line face to extend all the line.., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 41d6d91 04/15: Fixed extend face filtered for prop., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id d5ded9d 03/15: Conditional merged face to extend after eol., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 567644e 06/15: Add space for cursor to work also in terminal., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 9fa74c4 05/15: Changed handle_face_prop_general prototype., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 834e23f 09/15: Added parameter attr_filter to face_at_string_position., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id ae9787b 10/15: Extend_face simplify., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 19801c2 14/15: * etc/NEWS : Added comment about :extend face attribute in NEWS., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 6b50c01 13/15: Document new :extend face attribute in manuals., Jimmy Aguilar Mena, 2019/10/09
- [Emacs-diffs] feature/extend_face_id 0a7b2b0 15/15: ; extend_face_id and fill_column_indicator corrections,
Jimmy Aguilar Mena <=