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

[Boruch Baum <notifications@xxxxxxxxxx>]

• 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 'flat' 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 'fixed'. They weren'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: "FIXME: that the session-select
    window is selected at this time would probably be due to a bug."
    and "BUG: The final 'or' statement makes no sense, because
    w3m-fill-column is an integer, and if for some reason it had been
    set nil, then the '<' test above would have crashed the function"

---

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, view it on GitHub, or unsubscribe.