-
-
Notifications
You must be signed in to change notification settings - Fork 75
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Branch: refs/heads/master Date: 2017-02-20T21:46:15Z Author: Franco Pellegrini (frapell) <frapell@gmail.com> Commit: plone/plone.app.iterate@061b702 Add each field label for the diff view, fix visual diff link and add break Files changed: M CHANGES.rst M plone/app/iterate/browser/diff.pt Repository: plone.app.iterate Branch: refs/heads/master Date: 2017-02-21T07:21:50+01:00 Author: Jens W. Klein (jensens) <jk@kleinundpartner.at> Commit: plone/plone.app.iterate@5af9137 Merge pull request #41 from plone/diff_view Minor aesthetic changes for the diff view Files changed: M CHANGES.rst M plone/app/iterate/browser/diff.pt
- Loading branch information
Showing
1 changed file
with
128 additions
and
19 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,136 @@ | ||
Repository: mockup | ||
Repository: plone.app.iterate | ||
|
||
|
||
Branch: refs/heads/master | ||
Date: 2017-02-20T16:57:31-05:00 | ||
Author: esteele (esteele) <eric@esteele.net> | ||
Commit: https://github.com/plone/mockup/commit/66df926f9f36d9cf88e66a3c21f12e0b846d2a7b | ||
Date: 2017-02-20T21:46:15Z | ||
Author: Franco Pellegrini (frapell) <frapell@gmail.com> | ||
Commit: https://github.com/plone/plone.app.iterate/commit/061b7025900a14469eeb0dee2684742ca0c3b240 | ||
|
||
Fix breaking version number | ||
Add each field label for the diff view, fix visual diff link and add break | ||
|
||
Files changed: | ||
M package.json | ||
|
||
diff --git a/package.json b/package.json | ||
index 75a89ba..0f7e40c 100644 | ||
--- a/package.json | ||
+++ b/package.json | ||
@@ -1,6 +1,6 @@ | ||
{ | ||
"name": "mockup", | ||
- "version": "2.4.1.dev0", | ||
+ "version": "2.4.1", | ||
"description": "A collection of client side patterns for faster and easier web development", | ||
"homepage": "http://plone.github.io/mockup", | ||
"devDependencies": { | ||
M CHANGES.rst | ||
M plone/app/iterate/browser/diff.pt | ||
|
||
diff --git a/CHANGES.rst b/CHANGES.rst | ||
index 701375c..fa0bc9e 100644 | ||
--- a/CHANGES.rst | ||
+++ b/CHANGES.rst | ||
@@ -10,7 +10,8 @@ Breaking changes: | ||
|
||
New features: | ||
|
||
-- *add item here* | ||
+- Add each field label for the diff view, fix visual diff link and add break | ||
+ [frapell] | ||
|
||
Bug fixes: | ||
|
||
diff --git a/plone/app/iterate/browser/diff.pt b/plone/app/iterate/browser/diff.pt | ||
index dc795c9..b81e5be 100644 | ||
--- a/plone/app/iterate/browser/diff.pt | ||
+++ b/plone/app/iterate/browser/diff.pt | ||
@@ -38,12 +38,14 @@ | ||
tal:repeat="d diffs/getDiffs"> | ||
<div class="field" | ||
tal:condition="not:d/same" | ||
- tal:define="unified exists:d/unified_diff; | ||
+ tal:define="field_label d/label; | ||
+ unified exists:d/unified_diff; | ||
html exists:d/html_diff; | ||
inline exists:d/inline_diff; | ||
ndiff exists:d/ndiff; | ||
inline python: inline and not request.get('no_inline', 0); | ||
html python: html and not request.get('show_unified', 0)"> | ||
+ <label tal:content="field_label" /> | ||
|
||
<tal:html condition="inline"> | ||
<br /> | ||
@@ -58,7 +60,7 @@ | ||
<br /> | ||
<tal:visual-only tal:condition="exists:d/inline_diff"> | ||
<a href="#" | ||
- tal:attributes="href string:">Show visual differences</a> | ||
+ tal:attributes="href string:${diff_url}">Show visual differences</a> | ||
<br /> | ||
</tal:visual-only> | ||
<a href="#" | ||
@@ -66,6 +68,7 @@ | ||
<pre tal:content="structure d/html_diff">[html diff]</pre> | ||
</tal:html> | ||
<tal:unified condition="python:unified and not html"> | ||
+ <br /> | ||
<a href="#" | ||
tal:condition="exists:d/inline_diff" | ||
tal:attributes="href string:${diff_url}">Show visual differences</a> | ||
|
||
|
||
Repository: plone.app.iterate | ||
|
||
|
||
Branch: refs/heads/master | ||
Date: 2017-02-21T07:21:50+01:00 | ||
Author: Jens W. Klein (jensens) <jk@kleinundpartner.at> | ||
Commit: https://github.com/plone/plone.app.iterate/commit/5af9137778568370443dd4cf6deefe0bba6f7d2e | ||
|
||
Merge pull request #41 from plone/diff_view | ||
|
||
Minor aesthetic changes for the diff view | ||
|
||
Files changed: | ||
M CHANGES.rst | ||
M plone/app/iterate/browser/diff.pt | ||
|
||
diff --git a/CHANGES.rst b/CHANGES.rst | ||
index 701375c..fa0bc9e 100644 | ||
--- a/CHANGES.rst | ||
+++ b/CHANGES.rst | ||
@@ -10,7 +10,8 @@ Breaking changes: | ||
|
||
New features: | ||
|
||
-- *add item here* | ||
+- Add each field label for the diff view, fix visual diff link and add break | ||
+ [frapell] | ||
|
||
Bug fixes: | ||
|
||
diff --git a/plone/app/iterate/browser/diff.pt b/plone/app/iterate/browser/diff.pt | ||
index dc795c9..b81e5be 100644 | ||
--- a/plone/app/iterate/browser/diff.pt | ||
+++ b/plone/app/iterate/browser/diff.pt | ||
@@ -38,12 +38,14 @@ | ||
tal:repeat="d diffs/getDiffs"> | ||
<div class="field" | ||
tal:condition="not:d/same" | ||
- tal:define="unified exists:d/unified_diff; | ||
+ tal:define="field_label d/label; | ||
+ unified exists:d/unified_diff; | ||
html exists:d/html_diff; | ||
inline exists:d/inline_diff; | ||
ndiff exists:d/ndiff; | ||
inline python: inline and not request.get('no_inline', 0); | ||
html python: html and not request.get('show_unified', 0)"> | ||
+ <label tal:content="field_label" /> | ||
|
||
<tal:html condition="inline"> | ||
<br /> | ||
@@ -58,7 +60,7 @@ | ||
<br /> | ||
<tal:visual-only tal:condition="exists:d/inline_diff"> | ||
<a href="#" | ||
- tal:attributes="href string:">Show visual differences</a> | ||
+ tal:attributes="href string:${diff_url}">Show visual differences</a> | ||
<br /> | ||
</tal:visual-only> | ||
<a href="#" | ||
@@ -66,6 +68,7 @@ | ||
<pre tal:content="structure d/html_diff">[html diff]</pre> | ||
</tal:html> | ||
<tal:unified condition="python:unified and not html"> | ||
+ <br /> | ||
<a href="#" | ||
tal:condition="exists:d/inline_diff" | ||
tal:attributes="href string:${diff_url}">Show visual differences</a> | ||
|
||
|