Permalink
Browse files

MINOR Using HTML5 data attributes for denoting layout in templates

  • Loading branch information...
1 parent a196135 commit 4e99af70479abe3838c9501c33a18d5a3ffad36d @chillu chillu committed Feb 16, 2012
@@ -96,7 +96,7 @@ jQuery.noConflict();
// Move from inner to outer layouts. Some of the elements might not exist.
// Not all edit forms are layouted, so qualify by their data value.
- this.find('.cms-edit-form[data-layout]').redraw();
+ this.find('.cms-edit-form[data-layout-type]').redraw();
// Only redraw preview if its visible
var preview = this.find('.cms-preview');
@@ -1,4 +1,4 @@
-<div class="cms-content center" data-layout="{type: 'border'}" style="display: inline-block;">
+<div class="cms-content center" data-layout-type="border" style="display: inline-block;">
$Tools
@@ -1,5 +1,5 @@
<% if IncludeFormTag %>
-<form $FormAttributes data-layout="{type: 'border'}">
+<form $FormAttributes data-layout-type="border">
<% end_if %>
<div class="cms-content-header north">
<div>
@@ -1,4 +1,4 @@
-<div class="cms-menu cms-panel cms-panel-layout west" id="cms-menu" data-layout="{type: 'border'}">
+<div class="cms-menu cms-panel cms-panel-layout west" id="cms-menu" data-layout-type="border">
<div class="cms-logo-header north">
<div class="cms-logo">
<a href="http://www.silverstripe.org/" target="_blank" title="SilverStripe (Version - $CMSVersion)">
@@ -1,4 +1,4 @@
-<div class="cms-content center $BaseCSSClasses" data-layout="{type: 'border'}">
+<div class="cms-content center $BaseCSSClasses" data-layout-type="border">
<div class="cms-content-header north">
<div>
@@ -1,4 +1,4 @@
-<div class="cms-content-tools west cms-panel cms-panel-layout" data-expandOnClick="true" data-layout="{type: 'border'}">
+<div class="cms-content-tools west cms-panel cms-panel-layout" data-expandOnClick="true" data-layout-type="border">
<div class="cms-panel-content center">
<h3 class="cms-panel-header"><% _t('Filter', 'Filter') %></h3>
@@ -1,4 +1,4 @@
-<div class="cms-content center $BaseCSSClasses" data-layout="{type: 'border'}">
+<div class="cms-content center $BaseCSSClasses" data-layout-type="border">
<div class="cms-content-tools west">
<div class="cms-content-header north">
<div>
@@ -10,13 +10,13 @@
<% include CMSLoadingScreen %>
- <div class="cms-container center" data-layout="{type: 'border'}">
+ <div class="cms-container center" data-layout-type="border">
$Menu
$Content
- <div class="cms-preview east <% if IsPreviewExpanded %>is-expanded<% else %>is-collapsed<% end_if %>" data-layout="{type: 'border'}">
+ <div class="cms-preview east <% if IsPreviewExpanded %>is-expanded<% else %>is-collapsed<% end_if %>" data-layout-type="border">
<iframe src="about:blank" class="center" name="cms-preview-iframe"></iframe>
<div class="cms-preview-controls south"></div>
</div>

0 comments on commit 4e99af7

Please sign in to comment.