Mardown Help documentation not compatible with clean URLs
adrinux - November 24, 2004 - 15:59
| Project: | Markdown with SmartyPants |
| Version: | 4.5.x-1.x-dev |
| Component: | Documentation |
| Category: | bug report |
| Priority: | minor |
| Assigned: | teradome |
| Status: | closed |
Jump to:
Description
I can reach the actual 'Markdown Text Formatting Rules' page ok with clean URLs on, but the index at the top of the document has links to anchors as non-clean URLs (/?q=markdown/#blockquote), instead of the clean version (/markdown/#blockquote).
The workaround is to scroll or use browser 'find', but it would be nice if these links worked.
#1
This was a problem going the other way, re: http://drupal.org/node/10994
However, I've (finally) gone and put in a check for whether clean_urls is on or not and rewritten the base for the namerefs appropriately.
Please try the updated CVS version out -- I'm also trying to code around a problem with Drupal -- Drupal by default uses a base href tag, which prevents the use of named anchors without reloading the node/page. This is currently documented by this issue: http://drupal.org/node/4213
The approach I have -- forcing the initial link to use a nameref, seems to work to prevent the node from being reloaded by the browser in my testing...
#2
The following doesn't seem to work:
* * *
(in order to produce an horizontal line.
Also I tried to force a line break by strating a line with three spaces and the hitting return, but the break is simply ignored. Is the full markdown language implemented here? Many thanks!
#3
adriatichero: you should check to see if you have any other filters with higher priority than marksmarty -- there may be a conflict between them (i.e. the previous filter may be stripping the whitespace). any issues with markdown itself would be in the php implementation. your installed version number and links to the php-markdown project are on the marksmarty settings page. if you still have problems, please open a new bug report for it.
adrinux: considering that I've made it possible for people to upgrade the markdown and smartypants independent from the module itself, i've decided to have the CVS and 4.6 version link directly to the syntax documentation on the core markdown project site. It should be the same for 4.5 as well, though i will be double-checking that shortly.
#4
Re #1, there's a patch available that removes the base tag from pages generated by drupal. If that sounds interesting to you, you might want to comment on the bug so that it gets more attention: http://drupal.org/node/13148
#5
closed because of 1) age 2) applied patch 3) drupal 4.7 no longer uses BASE tag. hoorah.