* Bug: org-columns adds spaces [8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)]
@ 2015-07-13 11:34 Nicolas Richard
2015-07-13 12:52 ` Nicolas Richard
0 siblings, 1 reply; 3+ messages in thread
From: Nicolas Richard @ 2015-07-13 11:34 UTC (permalink / raw
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 26 bytes --]
Here's my test.org file :
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: test.org --]
[-- Type: text/x-org, Size: 164 bytes --]
* Test org
:PROPERTIES:
:COLUMNS: %15foo %fooo %2foooo %2fooooo %2foobar %2foobaz %2fooquw %2fooqux %2fooapp %2foopoj %2fooazf %2fooavp %10foopa %1whatever
:END:
[-- Attachment #3: Type: text/plain, Size: 649 bytes --]
Here's how I test it :
emacs -Q -L ~/sources/org-mode/lisp/ test.org -f whitespace-mode
then hit:
M-x whitespace-mode RET ;; not required
M-x outline-show-all RET ;; not required
C-c C-x C-c ;; enter org-columns
q ;; leave it
The buffer is still marked "unmodified" but there are five space
characters at the end of the headline. (and this will later confuse org
element cache which thus give errors, but I guess that's to be
expected.)
Emacs : GNU Emacs 25.0.50.1 (i686-pc-linux-gnu, X toolkit, Xaw scroll bars)
of 2015-07-11 on Aurora
Package: Org-mode version 8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Bug: org-columns adds spaces [8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)]
2015-07-13 11:34 Bug: org-columns adds spaces [8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)] Nicolas Richard
@ 2015-07-13 12:52 ` Nicolas Richard
2015-07-18 7:46 ` Nicolas Goaziou
0 siblings, 1 reply; 3+ messages in thread
From: Nicolas Richard @ 2015-07-13 12:52 UTC (permalink / raw
To: Nicolas Richard; +Cc: emacs-orgmode
Nicolas Richard <youngfrog@members.fsf.org> writes:
> The buffer is still marked "unmodified" but there are five space
> characters at the end of the headline. (and this will later confuse org
> element cache which thus give errors, but I guess that's to be
> expected.)
Meanwhile I had a look and it's due to this part of
org-columns-display-here :
(org-unmodified (insert " "))
(http://orgmode.org/cgit.cgi/org-mode.git/tree/lisp/org-colview.el?h=maint#n247)
Inserting characters in the buffer for the sake of columnview seems
wrong to me. Even more so if this is done silently.
I tried removing the whole (if then) block. That not only helps with the
spurious spaces, but also makes the column view work better in my
original usecase. OTOH I don't know what it will break.
--- /home/youngfrog/sources/org-mode/lisp/org-colview.el
+++ #<buffer org-colview.el>
@@ -224,13 +224,7 @@
(overlay-put ov 'org-columns-pom pom)
(overlay-put ov 'org-columns-format f)
(overlay-put ov 'line-prefix "")
- (overlay-put ov 'wrap-prefix ""))
- (if (or (not (char-after beg))
- (equal (char-after beg) ?\n))
- (let ((inhibit-read-only t))
- (save-excursion
- (goto-char beg)
- (org-unmodified (insert " ")))))) ;; FIXME: add props and remove later?
+ (overlay-put ov 'wrap-prefix "")))
;; Make the rest of the line disappear.
(org-unmodified
(setq ov (org-columns-new-overlay beg (point-at-eol)))
--
Nicolas Richard
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Bug: org-columns adds spaces [8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)]
2015-07-13 12:52 ` Nicolas Richard
@ 2015-07-18 7:46 ` Nicolas Goaziou
0 siblings, 0 replies; 3+ messages in thread
From: Nicolas Goaziou @ 2015-07-18 7:46 UTC (permalink / raw
To: Nicolas Richard; +Cc: emacs-orgmode
Hello,
Nicolas Richard <youngfrog@members.fsf.org> writes:
> Nicolas Richard <youngfrog@members.fsf.org> writes:
>> The buffer is still marked "unmodified" but there are five space
>> characters at the end of the headline. (and this will later confuse org
>> element cache which thus give errors, but I guess that's to be
>> expected.)
>
> Meanwhile I had a look and it's due to this part of
> org-columns-display-here :
> (org-unmodified (insert " "))
Thank you.
> Inserting characters in the buffer for the sake of columnview seems
> wrong to me.
Each column is an overlay applied on top of a character in the line.
Since columns cannot overlap, you need at least as many characters in
the line as columns to display.
> Even more so if this is done silently.
I fixed it in cf5cb15.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-07-18 7:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-13 11:34 Bug: org-columns adds spaces [8.3beta (release_8.3beta-1286-g20795f @ /home/youngfrog/sources/org-mode/lisp/)] Nicolas Richard
2015-07-13 12:52 ` Nicolas Richard
2015-07-18 7:46 ` Nicolas Goaziou
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.