Merge lp://staging/~dholbach/help-app/1430735 into lp://staging/~ubuntu-touch-coreapps-drivers/help-app/trunk

Proposed by Daniel Holbach
Status: Rejected
Rejected by: Daniel Holbach
Proposed branch: lp://staging/~dholbach/help-app/1430735
Merge into: lp://staging/~ubuntu-touch-coreapps-drivers/help-app/trunk
Diff against target: 9029 lines (+2726/-2425)
19 files modified
edit-here/content/pages/apps.md (+17/-15)
edit-here/content/pages/basic.md (+9/-8)
edit-here/content/pages/index.md (+2/-2)
edit-here/content/pages/scopes.md (+4/-3)
edit-here/content/pages/security.md (+8/-7)
edit-here/content/pages/settings.md (+8/-7)
edit-here/content/pages/store.md (+4/-3)
edit-here/content/pages/ui.md (+10/-9)
edit-here/pelicanconf.py (+9/-1)
edit-here/po/de.po (+392/-364)
edit-here/po/es.po (+394/-366)
edit-here/po/fr.po (+392/-364)
edit-here/po/help.pot (+171/-168)
edit-here/po/it.po (+392/-364)
edit-here/po/pt.po (+392/-367)
edit-here/po/ro.po (+392/-364)
edit-here/q-and-a.py (+77/-0)
edit-here/tests/test_translations.py (+6/-4)
edit-here/translations.py (+47/-9)
To merge this branch: bzr merge lp://staging/~dholbach/help-app/1430735
Reviewer Review Type Date Requested Status
David Planella Needs Fixing
Review via email: mp+252910@code.staging.launchpad.net

Commit message

This is a partial fix for bug 1430735.

What's happening here, is that we allow !!T and !!I as markup to indicate that a particular question/answer pair is either textonly (!!T) or include an image (!!I). This will add HTML markup through our our own q-and-a markdown extension.

We can now also use the attr_list markdown extension to show links as button. Merely add {: .button } at the end of a link.

Tests and stuff are updated as well.

What's not working yet:
 - !!I is implemented, but we need to figure out how to parse the image
   markup, so we can put it in its own <div>

To post a comment you must log in.
97. By Daniel Holbach

merge from trunk

98. By Daniel Holbach

remove superfluous print

99. By Daniel Holbach

remove stray print

100. By Daniel Holbach

add initial support for creating an answer-with-image block

Revision history for this message
David Planella (dpm) wrote :

I need to actually test the branch, but in the meantime I've looked at the code and added some comments.

review: Needs Fixing
101. By Daniel Holbach

remove button tags, they are not 'Call To Action buttons'

102. By Daniel Holbach

use .link-cta-ubuntu instead of .button

103. By Daniel Holbach

updated .pot and .po files accordingly

104. By Daniel Holbach

merge trunk

105. By Daniel Holbach

remove bzr diffI tag for now, it's not implemented yet and we won't need it for version1 in ths store

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Trying all the documented make targets:

make check: no errors
make click: no errors
make web: WARNING: Unable to find `/images/settings.gif`, skipping url replacement.
make html: WARNING: Unable to find `/images/settings.gif`, skipping url replacement.

106. By Daniel Holbach

add image

Revision history for this message
Daniel Holbach (dholbach) wrote :

Fixed.

107. By Daniel Holbach

add [TOC] to all FAQ pages, add T tag to all questions

108. By Daniel Holbach

make pep8 happy

109. By Daniel Holbach

remove attr_list (and other statements), if we find them in translated strings

110. By Daniel Holbach

simplify hide/readd logic somewhat

Revision history for this message
Daniel Holbach (dholbach) wrote :

Unfortunately translated links are broken again... I'll fix this tomorrow morning. It'd be nice though, if somebody could review the rest of the MP.

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Just a question -- can we assume text only for any question / answer pairs so we don't have to litter all our our content with !!T?

Revision history for this message
Daniel Holbach (dholbach) wrote :

How can Markdown know which part is a questions/answer block and which one isn't? :-/

Revision history for this message
Daniel Holbach (dholbach) wrote :

I will break this up into smaller portions as it will be easier to review...

Unmerged revisions

110. By Daniel Holbach

simplify hide/readd logic somewhat

109. By Daniel Holbach

remove attr_list (and other statements), if we find them in translated strings

108. By Daniel Holbach

make pep8 happy

107. By Daniel Holbach

add [TOC] to all FAQ pages, add T tag to all questions

106. By Daniel Holbach

add image

105. By Daniel Holbach

remove bzr diffI tag for now, it's not implemented yet and we won't need it for version1 in ths store

104. By Daniel Holbach

merge trunk

103. By Daniel Holbach

updated .pot and .po files accordingly

102. By Daniel Holbach

use .link-cta-ubuntu instead of .button

101. By Daniel Holbach

remove button tags, they are not 'Call To Action buttons'

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches