[emacs-w3m:13807] [emacs-w3m/emacs-w3m] force-wide-display (#98)

Boruch Baum <[email protected]> Wed, 28 Jul 2021 07:18:07 -0700
Newsgroups gmane.emacs.w3m
Message-ID <emacs-w3m/emacs-w3m/pull/[email protected]>
+ It is desirable for normal web pages to have a limited text width
  because beyond a certain width, it is difficult to return to the
  proper next line. However, the content of some pages does make sense
  to always take advantage of the full width of a window. This commit
  enables that distinction to be made, by url regex, and uses that
  feature for the &#39;flat&#39; history display pages, as created by M-x
  w3m-db-history.

+ The layout of w3m-db-history pages is reversed to put the short and
  constant width field (date/time) on the left.

+ Function w3m-about-db-history has some unusual code that directly
  calls function w3m-display-width instead of via w3m-create-page, so
  it was necessary there to manually set variable w3m-fill-column to
  -1. I tried several other techniques, but for any other to work
  involved even more ugly code.

+ If hl-line mode is available, in is enabled for pages that are
  forced wide.

+ Logic bugs in function w3m-display-width were &#39;fixed&#39;. They weren&#39;t
  breaking things, but they have been marked for years as nonsense
  lines with noone taking the courage to remove them.

  + There were two of these bugs: &quot;FIXME: that the session-select
    window is selected at this time would probably be due to a bug.&quot;
    and &quot;BUG: The final &#39;or&#39; statement makes no sense, because
    w3m-fill-column is an integer, and if for some reason it had been
    set nil, then the &#39;&lt;&#39; test above would have crashed the function&quot;
You can view, comment on, or merge this pull request online at:

  https://github.com/emacs-w3m/emacs-w3m/pull/98

-- Commit Summary --

  * force-wide-display

-- File Changes --

    M ChangeLog (8)
    M w3m.el (57)

-- Patch Links --

https://github.com/emacs-w3m/emacs-w3m/pull/98.patch
https://github.com/emacs-w3m/emacs-w3m/pull/98.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/emacs-w3m/emacs-w3m/pull/98