unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#6091: 23.1.96; fringe face can not be remapped
@ 2010-05-03 17:25 Leo
  2010-07-06 20:25 ` Chong Yidong
  2010-07-10  7:54 ` Leo
  0 siblings, 2 replies; 7+ messages in thread
From: Leo @ 2010-05-03 17:25 UTC (permalink / raw)
  To: 6091


It seems fringe is about the only face that can't be remapped. For
example, eval:

  (push '(fringe (:foreground "#71d3b4" :background
                  "#233323"))
        face-remapping-alist)

and you will see no change on the fringes.

Leo







^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-05-03 17:25 bug#6091: 23.1.96; fringe face can not be remapped Leo
@ 2010-07-06 20:25 ` Chong Yidong
  2010-07-07  0:08   ` Leo
  2010-07-10  7:54 ` Leo
  1 sibling, 1 reply; 7+ messages in thread
From: Chong Yidong @ 2010-07-06 20:25 UTC (permalink / raw)
  To: Leo; +Cc: 6091-done

> It seems fringe is about the only face that can't be remapped.

Should be fixed now in the trunk, thanks.





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-07-06 20:25 ` Chong Yidong
@ 2010-07-07  0:08   ` Leo
  2010-07-10 18:01     ` Chong Yidong
  0 siblings, 1 reply; 7+ messages in thread
From: Leo @ 2010-07-07  0:08 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 6091

[-- Attachment #1: Type: text/plain, Size: 221 bytes --]

On 2010-07-06 21:25 +0100, Chong Yidong wrote:
>> It seems fringe is about the only face that can't be remapped.
>
> Should be fixed now in the trunk, thanks.

The fix only worked partially. See the attached screenshot:


[-- Attachment #2: Screen shot 2010-07-07 at 00.56.34.png --]
[-- Type: image/png, Size: 63453 bytes --]

[-- Attachment #3: Type: text/plain, Size: 205 bytes --]


part of the fringes still use the old face.

BTW, the 'cursor' face seems to have a similar issue, i.e.

    (push '(cursor (:background "red")) face-remapping-alist)

does not do anything.

Thanks.

Leo

^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-05-03 17:25 bug#6091: 23.1.96; fringe face can not be remapped Leo
  2010-07-06 20:25 ` Chong Yidong
@ 2010-07-10  7:54 ` Leo
  1 sibling, 0 replies; 7+ messages in thread
From: Leo @ 2010-07-10  7:54 UTC (permalink / raw)
  To: 6091

reopen 6091

I have re-opened this bug for now.





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-07-07  0:08   ` Leo
@ 2010-07-10 18:01     ` Chong Yidong
  2010-07-11  5:15       ` Leo
  0 siblings, 1 reply; 7+ messages in thread
From: Chong Yidong @ 2010-07-10 18:01 UTC (permalink / raw)
  To: Leo; +Cc: 6091

Leo <sdl.web@gmail.com> writes:

> On 2010-07-06 21:25 +0100, Chong Yidong wrote:
>>> It seems fringe is about the only face that can't be remapped.
>>
>> Should be fixed now in the trunk, thanks.
>
> The fix only worked partially. See the attached screenshot:

I can't reproduce this.  Please provide a recipe.





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-07-10 18:01     ` Chong Yidong
@ 2010-07-11  5:15       ` Leo
  2010-07-11 15:59         ` Chong Yidong
  0 siblings, 1 reply; 7+ messages in thread
From: Leo @ 2010-07-11  5:15 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 6091

On 2010-07-10 19:01 +0100, Chong Yidong wrote:
>> The fix only worked partially. See the attached screenshot:
>
> I can't reproduce this.  Please provide a recipe.

1. emacs -Q
2. paste the following into *scratch*

  (push '(fringe (:foreground "#71d3b4" :background
                  "#233323"))
        face-remapping-alist)

3. C-j (not C-x C-e)

Leo





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#6091: 23.1.96; fringe face can not be remapped
  2010-07-11  5:15       ` Leo
@ 2010-07-11 15:59         ` Chong Yidong
  0 siblings, 0 replies; 7+ messages in thread
From: Chong Yidong @ 2010-07-11 15:59 UTC (permalink / raw)
  To: Leo; +Cc: 6091

Leo <sdl.web@gmail.com> writes:

> 1. emacs -Q
> 2. paste the following into *scratch*
>
>   (push '(fringe (:foreground "#71d3b4" :background
>                   "#233323"))
>         face-remapping-alist)
>
> 3. C-j (not C-x C-e)

This is because the Emacs redisplay engine doesn't repaint the parts of
the screen unless it has changed, and it has no way to know that you've
changed the variable `face-remapping-alist'.  You can order a refresh by
calling `redraw-display'.

We could set up a more elaborate system for the redisplay engine to keep
track of `face-remapping-alist', but I don't think it's warranted, and
it may slow down the much more common case where `face-remapping-alist'
never changes.  Instead, I will update the doc of `face-remapping-alist'
to mention this issue.





^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2010-07-11 15:59 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-03 17:25 bug#6091: 23.1.96; fringe face can not be remapped Leo
2010-07-06 20:25 ` Chong Yidong
2010-07-07  0:08   ` Leo
2010-07-10 18:01     ` Chong Yidong
2010-07-11  5:15       ` Leo
2010-07-11 15:59         ` Chong Yidong
2010-07-10  7:54 ` Leo

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).