freenode/#clim - IRC Chatlog
Search
11:36:39
scymtym
TEXT-EDITOR-GADGET accepts a :SCROLL-BARS initarg which it (supplied or not) passes to DREI-TEXT-EDITOR-SUBSTRATE. with the recent stream pane changes, this results in a warning at runtime in all cases. should we only forward the initarg if actually supplied or just not accept it?
13:40:30
ck_
what's the state of McCLIM development in general, can you summarize what the major construction sites are at the moment?
13:41:44
jackdaniel
input stream rewrite; command processing improvements; keystroke accelerators and menus improvements (this one); scroll gestures; new (pending) input editor
13:46:31
jackdaniel
this is certainly related to command processing, but I did not address this issue so far
13:46:36
scymtym
jackdaniel: could you have a look at #1072? it fixes the echoing issue and leaves prompting for later (the prompting part is very likely not a regression)
17:02:23
jackdaniel
scymtym: I've loaded mcclim from a branch translator-echo-return (87f26016) and loaded example code from the PR (with the presentation type thing)
17:06:19
scymtym
jackdaniel: i would believed you without proof. as i said, maybe it broke (although i tried it many times)
17:08:41
jackdaniel
sure, I've posted it because I might have misunderstood what is expected from the change
17:09:33
scymtym
right. with the change, it should still start a newline and print the prompt but the command itself should not be echoed
17:11:33
scymtym
ok, it is broken here as well. must the related to the last rebase (or it only ever worked accidentally and compiling with an empty cache just revealed that)
17:14:33
scymtym
jackdaniel: the reason is that i typed ":echo ni" in the attached example program :)
17:17:56
scymtym
just shows that we need a syntax-checking version of DEFINE-COMMAND and related macros
17:19:05
jackdaniel
and if ni is typed, a big animated knight who says ni appears, like the sl locomotive
17:19:08
jackdaniel
https://www.cyberciti.biz/tips/displays-animations-when-accidentally-you-type-sl-instead-of-ls.html