#3246: Please make the internal viewer open diffs and/or text/*
-------------------------------+--------------------------------------------
  Reporter:  antonio@…         |       Owner:  mutt-dev
      Type:  enhancement       |      Status:  new     
  Priority:  trivial           |   Milestone:          
 Component:  display           |     Version:  1.5.19  
Resolution:                    |    Keywords:          
-------------------------------+--------------------------------------------

Comment(by antonio@…):

 Important warning: I'm not a mutt-dev.

 Your patch does not convince me because you're checking AUTOVIEW and
 you're also checking if the auto_view option is set, the same things are
 also checked when mutt_needs_mailcap() is called (which is called by
 mutt_is_autoview()) so I believe that it doesn't make much sense to check
 those options twice.

 Your approach is also more oriented to whitelist some things for each
 auto_view is not needed but that skips thing like text/xml,
 text/rfc822-headers and some other things that you may want to visualize
 straight away and that mutt can easily show to you without problems.

 I would be more convinced by the patch proposed in #3496 but that also has
 the problem that formats like text/rtf would need an autoview entry.

 I'm still thinking about what patch would best suit the mutt Debian
 package.

-- 
Ticket URL: <http://dev.mutt.org/trac/ticket/3246#comment:8>
Mutt <http://www.mutt.org/>
The Mutt mail user agent

Reply via email to