changeset 1676:e86191684f4b beta

fixed some anchor id problems for changeset ranges
author Marcin Kuzminski <marcin@python-works.com>
date Sat, 12 Nov 2011 20:24:07 +0200
parents 7c487d2678c7
children 7276b170ce8b
files rhodecode/controllers/changeset.py rhodecode/lib/helpers.py rhodecode/public/css/diff.css rhodecode/templates/changeset/changeset.html
diffstat 4 files changed, 9 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/rhodecode/controllers/changeset.py	Sat Nov 12 17:30:12 2011 +0200
+++ b/rhodecode/controllers/changeset.py	Sat Nov 12 20:24:07 2011 +0200
@@ -268,10 +268,10 @@
         ccmodel = ChangesetCommentsModel()
 
         ccmodel.create(text=request.POST.get('text'),
-                       repo_id=c.rhodecode_db_repo.repo_id, 
-                       user_id=c.rhodecode_user.user_id, 
-                       revision=revision, f_path=request.POST.get('f_path'), 
-                       line_no = request.POST.get('line'))
+                       repo_id=c.rhodecode_db_repo.repo_id,
+                       user_id=c.rhodecode_user.user_id,
+                       revision=revision, f_path=request.POST.get('f_path'),
+                       line_no=request.POST.get('line'))
 
         return redirect(h.url('changeset_home', repo_name=repo_name,
                               revision=revision))
--- a/rhodecode/lib/helpers.py	Sat Nov 12 17:30:12 2011 +0200
+++ b/rhodecode/lib/helpers.py	Sat Nov 12 20:24:07 2011 +0200
@@ -33,7 +33,7 @@
 from webhelpers.date import time_ago_in_words
 from webhelpers.paginate import Page
 from webhelpers.html.tags import _set_input_attrs, _set_id_attr, \
-    convert_boolean_attrs, NotGiven
+    convert_boolean_attrs, NotGiven, _make_safe_id_component
 
 from vcs.utils.annotate import annotate_highlight
 from rhodecode.lib.utils import repo_name_slug
@@ -51,7 +51,7 @@
     return HTML.input(**attrs)
 
 reset = _reset
-
+safeid = _make_safe_id_component
 
 def get_token():
     """Return the current authentication token, creating one if one doesn't
--- a/rhodecode/public/css/diff.css	Sat Nov 12 17:30:12 2011 +0200
+++ b/rhodecode/public/css/diff.css	Sat Nov 12 20:24:07 2011 +0200
@@ -34,10 +34,6 @@
 	border-collapse: collapse;
 	width: 99%;
 }
-table.code-difftable td:target *{
-	background:  repeat scroll 0 0 #FFFFBE !important;
-	text-decoration: underline;
-}
 
 table.code-difftable td {
     padding: 0 !important; 
--- a/rhodecode/templates/changeset/changeset.html	Sat Nov 12 17:30:12 2011 +0200
+++ b/rhodecode/templates/changeset/changeset.html	Sat Nov 12 20:24:07 2011 +0200
@@ -63,7 +63,7 @@
 		             </div>                  
 		                 %if len(c.changeset.parents)>1:
 		                 <div class="merge">
-		                     ${_('merge')}<img alt="merge" src="${h.url("/images/icons/arrow_join.png")}"/>
+		                     ${_('merge')}<img alt="merge" src="${h.url('/images/icons/arrow_join.png')}"/>
 		                 </div>
 		                 %endif
 		                 
@@ -92,8 +92,7 @@
 	        <div class="cs_files">
 	                %for change,filenode,diff,cs1,cs2,stat in c.changes:
 	                    <div class="cs_${change}">
-		                    <div class="node">${h.link_to(h.safe_unicode(filenode.path),
-		                                        h.url.current(anchor=h.repo_name_slug('C%s' % h.safe_unicode(filenode.path))))}</div>
+		                    <div class="node">${h.link_to(h.safe_unicode(filenode.path),h.url.current(anchor='C-%s-%s' % (h.short_id(filenode.changeset.raw_id),h.safeid(h.safe_unicode(filenode.path)))))}</div>
 		                    <div class="changes">${h.fancy_file_stats(stat)}</div>
 	                    </div>
 	                %endfor
@@ -109,7 +108,7 @@
 		%if change !='removed':
 		<div style="clear:both;height:10px"></div>
 		<div class="diffblock  margined">
-			<div id="${h.repo_name_slug('C%s' % h.safe_unicode(filenode.path))}" class="code-header">
+			<div id="${'C-%s-%s' % (h.short_id(filenode.changeset.raw_id),h.safeid(h.safe_unicode(filenode.path)))}" class="code-header">
 				<div class="changeset_header">
 					<span class="changeset_file">
 						${h.link_to_if(change!='removed',h.safe_unicode(filenode.path),h.url('files_home',repo_name=c.repo_name,