The current whitespace faces make whitespace-mode too distracting to
read with too many background colors. This commit removes the annoying
colors for whitespace-space, whitespace-tab and whitespace-indentation
because:
- whitespace-tab and whitespace-space already use characters for
visualizing their appearances. whitespace-tab uses ">>" while
whitespace-space uses ".". Adding background colors is redundant and
annoying. Editors like Notepad++ or Vim doesn't use background colors
to depict such whitespace characters.
- According to the documentation of whitespace-indentation:
"Symbol face used to visualize 8 or more SPACEs at beginning of line.
Used when `whitespace-style' includes the value `indentation'."
It is used for highlighting first 8 characters or some customized
value. We must also disable its background colors otherwise our buffers
are still filled with annoying colors all over the places, since often
indentation is around 8 spaces or less. Anyway, with whitespace-space
and whitesspace-tab, such highlighting like whitespace-indentation is
redundant and not needed.
By using font-lock-comment-face, it's less distracting than the default
bright red, since comment face is designed to be subtle.
Also allow users to turn it off by adding an option
spacemacs-show-trailing-whitespace.
Originally `load-ess-on-demand` used `use-package` for the purposes of
actually loading `ess-site` and related elisp for dealing with R. But
since `use-package` doesn't return true or false when a package is
actually successfully loaded anymore, and the semantics of how it will
work aren't precisely clear, it makes sense to use `require` here
instead.
(defun load-ess-on-demand ()
(interactive)
(-all? '---truthy? (list
(require 'ess-site)
(require 'ess-R-object-popup)
(require 'ess-R-data-view))))
All the normal hooks setup by `use-package` will work as normal. I
actually don't see a reason we should use `use-package` instead of
`require` in this specific instance. Since `use-package` often defers
loading packages, it's arguably clearer to use `require` in this
particular instance.
Currently by default company knows when to flip the popup. However, when
the above option is set to t, it causes a strange behaviour: when the
tooltip is flipped, press down (i.e. M-n or arrow key) go up and press
up (i.e. M-p or arrow key) go down.
It is supposed to open plain markdown file inside Emacs when a prefix
argument is supplied and render the markdown file otherwise. Currently,
both actions open the markdown file in a rendered buffer. This commit
fixed the issue.
With current configuration, .h file is only associated with c++-mode
only when cc-mode is loaded. So, when a .h file is opened the first
time, it is opened and is applied with c-mode. Only the 2nd time then
c++-mode is properly used.
- RET is not used anymore
- TAB complete the common prefix and cycle between candidates
- To complete the selection use C-l or `jk` pressed quickly
- Yasnippet expand uses hippie-expand and M-/ or C-p
