Fix getAnnotatedRange to return propper contained range.

Fixed tests

Change-Id: I305eb81644fb3c804455b4fc13bdabcce97ea471
This commit is contained in:
Rob Moen 2012-05-17 13:17:18 -07:00
parent 3eb31dae09
commit a937ad22f7
2 changed files with 3 additions and 4 deletions

View file

@ -412,11 +412,10 @@ ve.dm.Document.prototype.getAnnotatedRangeFromOffset = function ( offset, annota
}
}
while ( end < this.data.length ) {
end++;
if ( this.offsetContainsAnnotation(end, annotation ) === false ) {
end--;
break;
}
end++;
}
return new ve.Range( start, end );
};

View file

@ -382,7 +382,7 @@ test( 'getAnnotatedRangeFromOffset', 1, function(){
],
'annotation': { 'type': 'bold' },
'offset': 3,
'expected': new ve.Range( 1, 4 )
'expected': new ve.Range( 1, 5 )
},
{
'msg': 'a linked',
@ -410,7 +410,7 @@ test( 'getAnnotatedRangeFromOffset', 1, function(){
],
'annotation': { 'type': 'link/internal' },
'offset': 3,
'expected': new ve.Range( 3, 6 )
'expected': new ve.Range( 3, 7 )
}
];