[debbugs-tracker] bug#14950: closed (24.3.50; gnus: Fontification of URLs is not satisfactory)

[GNU bug Tracking System]

Your message dated Fri, 26 Jul 2013 08:40:56 +0900
with message-id <address@hidden>
and subject line Re: bug#14950: 24.3.50; gnus: Fontification of URLs is not 
satisfactory
has caused the debbugs.gnu.org bug report #14950,
regarding 24.3.50; gnus: Fontification of URLs is not satisfactory
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
14950: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14950
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: 24.3.50; gnus: Fontification of URLs is not satisfactory Date: Thu, 25 Jul 2013 15:41:57 +0530

Gnus is from Emacs repo.

Within Gnus, 

  [Chicago Export filters for Org-mode]
  
http://repo.or.cz/w/JabRefChicagoForOrgmode.git/blob_plain/HEAD:/net.sf.jabref.export.Chicago.ODF(English)-1.2.jar


How does the above URL below. Is it partly in bold and partly in default
font.  (See attached screenshot) Specifically, the URL is fontified only
till "1" (but doesn't include trailing ".2.jar" part).


,---- How "1" is fontified
| Character code properties: customize what to show
|   name: DIGIT ONE
|   general-category: Nd (Number, Decimal Digit)
|   decomposition: (49) ('1')
| 
| This is an active area of a link (widget)Top.
| 
| 
| There are 2 overlays here:
|  From 4589 to 4697
|   face                 gnus-button
|  From 4589 to 4697
|   button               link (widget)Top
|   evaporate            t
|   face                 widget-button
|   follow-link          mouse-face
|   help-echo            "Follow the link"
|   keymap               [Show]
|   mouse-face           (highlight)
|   pointer              hand
| 
| 
| There are text properties here:
|   gnus-callback        gnus-button-push
|   gnus-data            [Show]
|   gnus-string          [Show]
|   mouse-face           highlight
`----

,---- How "2" is fontified
| Character code properties: customize what to show
|   name: DIGIT TWO
|   general-category: Nd (Number, Decimal Digit)
|   decomposition: (50) ('2')
`----

In GNU Emacs 24.3.50.23 (i686-pc-linux-gnu, GTK+ Version 2.20.1)
 of 2013-07-25 on debian-6.05
Bzr revision: 113532 address@hidden
Windowing system distributor `The X.Org Foundation', version 11.0.10707000
Important settings:
  value of $LANG: en_IN
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t

Major mode: Article

gnus-url-fontification-is-bad.png
Description: PNG image

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#14950: 24.3.50; gnus: Fontification of URLs is not satisfactory
>
>
>
>
> Date: 
>
> Fri, 26 Jul 2013 08:40:56 +0900
>
>
>
>
> User-agent: 
>
> Gnus/5.130008 (真 Gnus v0.8) Emacs/24.3.50 (i686-pc-cygwin)
>
>
>
>
> Andreas Schwab wrote:
> > Katsumi Yamaoka <address@hidden> writes:
>
> >> What should be improved is the default value of the user option
> >> `gnus-button-url-regexp'.  But it is too complicated to let me
> >> encourage to do it without enbugging.  Anyone?
>
> > How about this?
>
> > diff --git a/lisp/gnus/gnus-art.el b/lisp/gnus/gnus-art.el
> > index 31a108a..2433db2 100644
> > --- a/lisp/gnus/gnus-art.el
> > +++ b/lisp/gnus/gnus-art.el
> > @@ -7177,7 +7177,8 @@ groups."
> >         "\\(?:"
> >         ;; Match paired parentheses, e.g. in Wikipedia URLs:
> >         ;; http://thread.gmane.org/address@hidden
> > - "[" chars punct "]+" "(" "[" chars punct "]+" "[" chars "]*)" "["
> > chars "]*"
> > +       "[" chars punct "]+" "(" "[" chars punct "]+" "[" chars "]*)"
> > +       "\\(?:" "[" chars punct "]+" "[" chars "]" "\\)?"
> >         "\\|"
> >         "[" chars punct     "]+" "[" chars "]"
> >         "\\)"))
>
> Great!  I understood what what you changed meant.  Thanks.
> The original one only matches the last part of a url like:
>
> http://.../abc.def(ghi)jklmno
>
> But yours matches:
>
> http://.../abc.def(ghi)jkl.mno
>
> Maybe making it match the one like
>
> http://.../abc.def(ghi)jkl.mno/pqr(stu)vwx
>
> is not necessary.  I've installed your patch.
>
>
> --- End Message ---