]> sipb.mit.edu Git - ikiwiki.git/blobdiff - doc/todo/do_not_make_links_backwards.mdwn
(no commit message)
[ikiwiki.git] / doc / todo / do_not_make_links_backwards.mdwn
index 8725d7feb84dc369b8d8f621789944f48de18af1..d7ce2780689d017d1f6b38ed54ba74a29a612b46 100644 (file)
@@ -14,8 +14,9 @@ I started looking into this, after encouraging words from Joey ("very long term
 The following needs to be done:
 
  1. the `link_regexp` variable needs to be turned backwards (or frontwards, if you like :P) (./) added an option for this, working!
 The following needs to be done:
 
  1. the `link_regexp` variable needs to be turned backwards (or frontwards, if you like :P) (./) added an option for this, working!
- 2. a config setting need to be added to the `link` plugin so that we can choose if we want backwards links or not (./) `backwards_links`, how does that sound?
- 3. a (solid!) parser needs to be written for [[ikiwiki-transition]] to change the actual links (if necessary) (!) in progress, almost done
- 4. rewrite tests to take into account the two syntaxes
+ 2. a config setting need to be added to the `link` plugin so that we can choose if we want backwards links or not (./) `links_direction`, how does that sound? I have changed that from `backwards_links` to be more neutral. 'rtl' means `\[[link|text]]` and 'ltr' means `\[[text|link]]`
+ 3. a (solid!) parser needs to be written for [[ikiwiki-transition]] to change the actual links (if necessary) (./) done!
+ 4. rewrite tests to take into account the two syntaxes (!) I would need help here, always have trouble with unit tests...
+ 5. deal with underlays (!!)
 
 
-There's a blocker: we can't have a per-wiki backwards_links option, because of the underlay, common to all wikis, which needs to be converted. So the option doesn't make much sense. Not sure how to deal with this... Maybe this needs to be at the package level?
+<!> There's a caveat: we can't have a per-wiki backwards_links option, because of the underlay, common to all wikis, which needs to be converted. So the option doesn't make much sense. Not sure how to deal with this... Maybe this needs to be at the package level?