Merge commit 'e12b7f5e54730325c751a889ed2e08580b5ef6ba'
authorJoey Hess <joey@gnu.kitenet.net>
Sat, 11 Jul 2009 19:52:51 +0000 (15:52 -0400)
committerJoey Hess <joey@gnu.kitenet.net>
Sat, 11 Jul 2009 19:52:51 +0000 (15:52 -0400)
IkiWiki/Plugin/img.pm
debian/changelog
doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
doc/bugs/nested_raw_included_inlines.mdwn
doc/bugs/openid_no_longer_pretty-prints_OpenIDs.mdwn
doc/bugs/prettydate_with_weekday-date_inconsistency.mdwn
doc/forum/Can_OpenID_users_be_adminusers__63__.mdwn
doc/ikiwiki/directive/img.mdwn
doc/plugins/anonok.mdwn
doc/todo/pretty-print_OpenIDs_even_if_not_enabled.mdwn [new file with mode: 0644]

index a186abdfce81f80ab8625d6c6b05872b34167c7c..78e3789171c53a5e90110f666e08ef0860bd36f7 100644 (file)
@@ -123,6 +123,7 @@ sub preprocess (@) {
                '" height="'.$im->Get("height").'"'.
                (exists $params{alt} ? ' alt="'.$params{alt}.'"' : '').
                (exists $params{title} ? ' title="'.$params{title}.'"' : '').
+               (exists $params{align} ? ' align="'.$params{align}.'"' : '').
                (exists $params{class} ? ' class="'.$params{class}.'"' : '').
                (exists $params{id} ? ' id="'.$params{id}.'"' : '').
                ' />';
index e83e570d315874f82314bbe1408faf8fa5ef9060..5001031df0f52098fbc8b8cfd3bc7bb6cd5e81a8 100644 (file)
@@ -1,6 +1,7 @@
 ikiwiki (3.1416) UNRELEASED; urgency=low
 
   * svn: Fix rcs_rename to properly scope call to dirname.
+  * img: Pass the align parameter through to the generated img tag.
 
  -- Joey Hess <joeyh@debian.org>  Wed, 08 Jul 2009 13:10:38 -0400
 
index b1f8ed2b31cbcc6d08bcfcb40ed45225a8bbe1ea..4ce257252fd96114ac5aedca9c4c2f5c78138c81 100644 (file)
@@ -33,3 +33,8 @@ Patch[[!tag patch]]:
      
 
 -- [[Jon]]
+
+> Strictly speaking, a `<ul>` with no `<li>`s isn't valid HTML either...
+> could `map` instead delay emitting the first `<ul>` until it determines that
+> it will have at least one item? Perhaps refactoring that function into
+> something easier to regression-test would be useful. --[[smcv]]
index 792bc843cf9b65bebe7af51c778d759a5e62063f..33433e235de7330b39c3d2949128d8c6f9217329 100644 (file)
@@ -24,3 +24,11 @@ In my real world situation, page1 is actually listing all pages that match a cer
 Whenever a page got tagged, it will appear on page1 but not on page0.
 
 Am I missing something? Is this a bug or Ikiwiki not supposed to support this use case?
+
+> Perhaps the inline plugin isn't being clever enough about dependencies -
+> strictly speaking, when a page is inlined with full content, the inlining
+> page should probably inherit all the inlined page's dependencies.
+> That might be prohibitively slow in practise due to the way IkiWiki
+> currently merges pagespecs, though - maybe the patches I suggested for
+> [[separating_and_uniquifying_pagespecs|todo/should_optimise_pagespecs]]
+> would help? --[[smcv]]
index 1e2030b23458cfee1d52441157a83f01085934f0..85a206bc0854bcfe59fb248c83a35b699241de95 100644 (file)
@@ -8,22 +8,10 @@ The git commit (in my `openid` branch) says it all:
 
 This bug affects ikiwiki.info (my commits show up in [[RecentChanges]] as http://smcv.pseudorandom.co.uk/ rather than smcv [pseudorandom.co.uk]).
 
-> Cherry picked, thanks. --[[Joey]] 
+> Cherry picked, thanks. --[[Joey]]
 
-Relatedly, the other commit on the same branch would be nice to have:
+Relatedly, the other commit on the same branch would be nice to have
+(edited to add: I've now moved it, and its discussion, to
+[[todo/pretty-print_OpenIDs_even_if_not_enabled]]). --[[smcv]]
 
-    Allow the openid plugin to be loaded but disabled, for its side-effect of defining IkiWiki::openiduser
-    
-    On various sites I have two IkiWiki instances running from the same
-    repository: one accessible via http and only accepting openid logins,
-    and one accessible via authenticated https and only accepting httpauth.
-    Ideally, the https version should still pretty-print OpenIDs seen in
-    git history.
-
---[[smcv]]
-
-> I wonder if an option is the best approach. Maybe it would be better to
-> simply move `openiduser` into `userlink`, and thus always support openid
-> usernames whether the plugin is enabled or not. --[[Joey]] 
-
-[[!tag patch]]
+[[!tag done]]
index 111ade8dd28675ea79201c83f97d8cf9dda8c504..430d65a3fa45d30d0c1c9085f9dd6b3559c7fe23 100644 (file)
@@ -28,3 +28,5 @@ Prettydate creates strings like this: _Last edited in the wee hours of Tuesday n
 > --[[Joey]]
 
 >> fair enough, I think I can get converted to a warped time perspective. --ulrik
+
+>>> Perhaps we can consider this [[done]], then? --[[smcv]]
index ca80c93f4ab4ccf643f3c9d475ce6202068d0425..7599e71e5c10ce60a6e3bd8032cb2c69bb95882d 100644 (file)
@@ -65,3 +65,5 @@ index 0bf100a..77b467a 100644
 >>>> If the end if "1" you should see the "Wiki Setup" button, if not the
 >>>> problem is not in determining if you're an admin, but elsewhere..
 >>>> --[[Joey]]
+
+I was being incredibly stupid and missed that websetup is a **plugin** and thus needed to be enabled.  Many thanks for your patient assistance, by helping me eliminate the unlikely it eventually led me to the obvious.  Cheers.  -- [[Adam]]
index 1d1f29bea875102f1c3973cf081ed8ba36f9274f..66efd008e47db04b8c2b8313b3f010a5ef6596e2 100644 (file)
@@ -18,9 +18,9 @@ making the image smaller than the specified size. You can also specify only
 the width or the height, and the other value will be calculated based on
 it: "200x", "x200"
 
-You can also pass `alt`, `title`, `class` and `id` parameters. These are
-passed through unchanged to the html img tag. If you include a `caption`
-parameter, the caption will be displayed centered beneath the image.
+You can also pass `alt`, `title`, `class`, `align` and `id` parameters.
+These are passed through unchanged to the html img tag. If you include a
+`caption` parameter, the caption will be displayed centered beneath the image.
 
 The `link` parameter is used to control whether the scaled down image links
 to the full size version. By default it does; set "link=somepage" to link
index a3fec4d8990d8bb72afd6ea155e3a3d0c66d839e..497ca07c8d0e49a7704459ab6e025a793d980a7c 100644 (file)
@@ -6,7 +6,7 @@ anonymous web users, who have not signed in, to edit any page in the wiki
 by default.
 
 The plugin also has a configuration setting, `anonok_pagespec`. This
-[[PageSpec]] can be used to allow anonymous editing of matching pages.
+[[ikiwiki/PageSpec]] can be used to allow anonymous editing of matching pages.
 
 If you're using the [[comments]] plugin, you can allow anonymous comments
 to be posted by setting:
diff --git a/doc/todo/pretty-print_OpenIDs_even_if_not_enabled.mdwn b/doc/todo/pretty-print_OpenIDs_even_if_not_enabled.mdwn
new file mode 100644 (file)
index 0000000..373c120
--- /dev/null
@@ -0,0 +1,24 @@
+A feature I originally requested on
+[[a_related_bug|bugs/openid_no_longer_pretty-prints_OpenIDs]]:
+
+    Allow the openid plugin to be loaded but disabled, for its side-effect of defining IkiWiki::openiduser
+    
+    On various sites I have two IkiWiki instances running from the same
+    repository: one accessible via http and only accepting openid logins,
+    and one accessible via authenticated https and only accepting httpauth.
+    Ideally, the https version should still pretty-print OpenIDs seen in
+    git history.
+
+--[[smcv]]
+
+> I wonder if an option is the best approach. Maybe it would be better to
+> simply move `openiduser` into `userlink`, and thus always support openid
+> usernames whether the plugin is enabled or not. --[[Joey]]
+
+>> OK, implemented that as 'smcv/always-openid'; if you don't think that's
+>> bloating the IkiWiki core too much, please consider merging. The poll on
+>> [[news/openid]] indicates fairly strong support for *only* accepting OpenID
+>> logins, so I think recognising OpenIDs can reasonably be considered core
+>> functionality! --[[smcv]]
+
+[[!tag patch]]