[PATCH conductor 7/7] remove unused templates
Mo Morsi
mmorsi at redhat.com
Wed May 9 10:35:28 UTC 2012
On 05/09/2012 11:20 AM, Jirka Tomasek wrote:
> On 05/03/2012 12:46 PM, Mo Morsi wrote:
>> ---
>> src/app/views/layouts/_tabpanel.html.haml | 20
>> --------------------
>> src/app/views/users/_section_header.html.haml | 7 -------
>> 2 files changed, 0 insertions(+), 27 deletions(-)
>> delete mode 100644 src/app/views/layouts/_tabpanel.html.haml
>> delete mode 100644 src/app/views/users/_section_header.html.haml
>>
>> diff --git a/src/app/views/layouts/_tabpanel.html.haml
>> b/src/app/views/layouts/_tabpanel.html.haml
>> deleted file mode 100644
>> index b603893..0000000
>> --- a/src/app/views/layouts/_tabpanel.html.haml
>> +++ /dev/null
>> @@ -1,20 +0,0 @@
>> -%header#tab-container-1
>> - .section-controls
>> - -#%a.collapse{'href'=>'#'} Collapse
>> - %nav.faceted
>> - %ul#tab-container-1-nav
>> - - @tabs.each do |tab|
>> - %li
>> - %a{ :href => url_for(:details_tab => tab[:id], :view =>
>> "filter", :only_tab => true),
>> - :id => "details_#{tab[:id]}", :class => "#{'active'
>> if @details_tab[:id] == tab[:id]}",
>> - 'data-pretty_view_toggle' => tab[:pretty_view_toggle] }
>> - = tab[:name]
>> -.content.collapsible#tab
>> - = render :partial => @details_tab[:view]
>> -
>> -:javascript
>> - $(document).ready(function() {
>> - $("#tab-container-1-nav li a").click(function(){
>> - Conductor.tabIsClickedResetFilters = true;
>> - });
>> - });
>> diff --git a/src/app/views/users/_section_header.html.haml
>> b/src/app/views/users/_section_header.html.haml
>> deleted file mode 100644
>> index c798674..0000000
>> --- a/src/app/views/users/_section_header.html.haml
>> +++ /dev/null
>> @@ -1,7 +0,0 @@
>> -%header.page-header
>> - .obj_actions
>> - .return_to
>> - =t'users.quick_jump'
>> - = link_to 'Groups', "#groups"
>> - = link_to 'Users', "#users"
>> - = link_to 'Roles', "#roles"
>> \ No newline at end of file
> The _tabpanel partial is used in pools_controller, so It should not be
> removed probably
>
> Jirka
Ah good catch, not sure how I missed this. Will just omit this removal
from subsequent patchsets.
-Mo
More information about the aeolus-devel
mailing list