Age | Commit message (Collapse) | Author |
|
Well, that was a PITA.
Luckily, this doesn't break guids to comments in rss feeds,
though it does change the links.
I haven't put in a warning about needing to rebuild to get
this fix. It's probably good enough for new comments to get the
fix, without a lot of mass rebuilding.
|
|
|
|
|
|
|
|
Add pageheader div so that misc.tmpl is consistent with page.tmpl. ---
templates/misc.tmpl | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
|
|
|
|
|
|
Remove actions from it, and avoid a broken title link.
|
|
|
|
|
|
* repolist: New plugin to support the rel=vcs-* microformat.
* goodstuff: Include repolist by default. (But it does nothing until
configured with the repository locations.)
|
|
|
|
|
|
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.
|
|
|
|
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.
|
|
|
|
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).
|
|
COMMENTAUTHORURL
Also refactor page.tmpl to use if/else rather than unless/if.
|
|
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.
|
|
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.
|
|
|
|
|
|
|
|
|
|
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.)
|
|
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.
|
|
This link will supplant the usual Discussion link for pages
that have comments enabled.
|
|
|
|
to claimed author if unauthenticated
This still isn't quite right (it displays the IP address twice if the user
doesn't specify a name).
|
|
|
|
Also provide a way for the comment template to pick up the verified
username/IP.
|
|
This has the side-effect that Ikiwiki's default style.css gives the text
box 100% width.
|
|
|
|
This delays all comment formatting until the last possible time, allows
us to set metadata without worrying that commenters may be able to evade
it, and means that changes to how a comment is saved can be handled
gracefully. It also gives us somewhere to put the commenter's username
or IP address for later reference.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Now that posts are individually sanitized, that should be safe.
|
|
The PageSpec is still called "postcomment" since that's what it means.
|
|
|
|
Also put "posting comments disabled" in [], and change "Page preview"
to "Comment preview".
|
|
|
|
|
|
smcvpostcomment)
|
|
Can't escape things to entities if the template then escapes the entities.
(aggregate doesn't have this problem.)
|
|
earlier added to edit links.
|