unify pagespecs

fix links

formatting

mention comments plugin

avoid an uninitialized value warning

openid branch merged

Merge commit 'smcv/openid'

more reorg

close merged bugs

add another test

Merge commit 'smcv/comments'

Merge branch 'comments'

Another fixed bug in beautify_urlpath

trivial bug with patch

Another comments improvement

rssitem.tmpl: include comments URL (HTML only)

atomitem.tmpl: include comments URLs (HTML and Atom versions)

comments: substitute commentsurl and atomcommentsurl for use in feeds

Mention a patch to improve OpenID display

Add a regression test for &openiduser

openid: in &openiduser, allow subdirectory-style providers to end with '/'

This improves the display of OpenIDs like 'http://id.mayfirst.org/jamie/'
(taking an example from the IkiWiki commit log).

openid: in &openiduser, let domain-style providers have arbitrarily many subdomains

This leads to better display for OpenIDs like smcv.pseudorandom.co.uk
and thm.id.fedoraproject.org (to take a couple of examples from the
IkiWiki commit history).

Record another couple of pending bugfixes

beautify_urlpath: add a regression test

beautify_urlpath: if path already starts with ./, don't add another ./

beautify_urlpath: add ./ before checking for /index.html

This fixes a bug: when a page links to its own #comments anchor you would
get a link like "index.html#comments" rather than "./#comments".

comments: run pagetemplate hooks

This fixes the bug that comments are always said to be from an anonymous
user at an unknown IP address.

comments: linkify and preprocess preview with correct 'page' param

Merge commit 'origin/comments' into comments-joey

todo/comments: Properly escape inline HTML

todo/comments: reorganise by status

comments: note fixed things and recommend merging origin/comments

fix markdown

meta: Process meta date during scan pass so that the date will always affect sorting in inlines.

analysis

comment layout change

I saw a layout similar to this on blogger, and I sorta like it

The dash avoids parens sitting next to each other in some cases.

Merge branch 'master' into comments

Response; added shell code and html formating to reproduction steps

avoid storing transient state in pagestate

None of the comment state needs to be stored through the a later run of
ikiwiki, so move it all from pagestate to a more transient storage.

This is assuming that we'll never want to add pagespecs to search against
the comment state. Pagespecs like author() are why the meta plugin does
store its meta data in pagestate -- the data can be needed later to match
against.

response

Sort paramater not a problem; Revised steps to reprodce

alternate idea

remove signin icons, use title

tweak author display

Put the icon after the name, mostly because it scans better on
non-graphical browsers where the alt text is displayed. And because the
name is really the more important part.

my try at improving comment styling

Makes it look more like a blog, but not enough to be confusing, and with
nothing as large as in a blog. Removal of the vertical line under the
subject imho makes it easier to scan through comments as each box is a new
one. Bolding the subject seems to make it stand out enough, especially as
its a link now. (Also considered increasing its font size to 110%.)

note new bug

response

Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info

Workaround and possible cause

Merge commit 'smcv/comments'

Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info

comment.tmpl: make anon/OpenID/signed-in icons independent of smileys

comments_display: display (?) for anon users, {x} for OpenIDs and {*} for local logins

This is a mockup of Joey's idea; to do it properly, the icons should
move to the basewiki or to a comments underlay, and {x} should be
replaced with an OpenID logo (if one with clear licensing even exists).

comments: Rename COMMENTURL to ADDCOMMENTURL to avoid confusion with COMMENTAUTHORURL

Also refactor page.tmpl to use if/else rather than unless/if.

comments: pass COMMENTOPENID to templates

comments: remove linkuser(), it's been integrated into preprocess() now

_comment directive: if the user looks like an OpenID, store that

haskell

note

minor edits

added details on cgi wrapper creation

initial page

vimperator can do this, too.

avoid uninitialized value

This can happen when a new field,
such as the new lasttry, is added.

fix comment permalink to always point to comment parent page

update

typos

simplify and compact the comment edit form

Mostly to make it more visually similar to the page edit form.

I'm a bit uncertian about the placement of the page type selector,
and about removing the "Page type". May rethink that.

remove 'signed in as foo'

I think users should know who they're signed in as; the edit pages don't
say so there's no reason to here. Also, the user id was not displayed
pretty.

add link from comment subject to its permalink

fiddle with comment css classes

remove cruft

wtf does it do? absolutely nothing

rename comments_form to editcomment

rename comments_display to comment

replace discussion links on pages with comments link

The thinking here is that having both a Discussion page and comments for
the same page is redundant, and certianly not what you want if you enable
comments for a page. At first I considered making configurable via pagespec
what pages got discussion links. But that would mean testing a new pagespec
for every page, and a redundant config setting to keep in sync. So intead,
take a lead from my previous change to make inlined pages have a comments
link, and change the discussion link at the top of regular pages to link to
their comments.

(Implementation is a bit optimised to avoid redundant pagespec checking.)

updated spanish translation

jump to comment after posting

Jumping to the just posted comment was the imputus, but I killed a number
of birds here.

Added a INLINEPAGE template variable, which can be used to add anchors to
any inline template.

To keep that sufficiently general, it is the full page name, so the
comment anchors and links changed form.

Got rid of the FIXMEd hardcoded html anchor div.

More importantly, the anchor is now to the very top of the comment, not the
text below. So you can see the title, and how it attributes you.

Avoid changing the permalink of pages that are not really comments, but
happen to contain the _comment directive. I think that behavior was a bug,
though not a likely one to occur since _comment should only really be used
on comment pages.

document how to make a global comments feed

responses

Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info

Update todo/comments with today's fixes, and thoughts about the remaining items

clarify

pagespec thoughts

note about comments feed

aggregate: If a feed fails to be downloaded, try again immediatly next time aggregation is run, even if the usual time has not passed. Closes: #508622 (Michael Gold)

close

deletion

jump to comments anchor after post

Not ideal, it would be nicer to jump to the actual comment posted, but no
anchor is available.

typo

fix

simplify pagespec

two comments ui suggestions

add Comments link when displaying a page inline

This link will supplant the usual Discussion link for pages
that have comments enabled.

fix test when comments_closed_pagespec is empty

don't rely on $@

A malformed pagespec will cause $@ to be set when translated, but if
it is used a second time, the memoization will defeat that check. Better to
check for the result not being defined.

change around comments pagespecs

I think it is clearer to have one pagespec that controls all pages with
comments, and a separate pagespec that can be used to close new comments on
a subset of those pages.

fix default values for config settings

defaults cannot be set in getsetup, do it in checkconfig to avoid
uninitialized value warnings.

clarification