jquery.textSelection: Remove outdated FIXME comments
authorBartosz Dziewoński <matma.rex@gmail.com>
Tue, 30 Jan 2018 09:26:15 +0000 (10:26 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Tue, 30 Jan 2018 09:26:15 +0000 (10:26 +0100)
These are from 2010 or earlier; they blame to 32377424b94 / rSVN72349,
which is a huge SVN branch merge and I'm not going to dig past that.

To be honest I don't really understand them (perhaps some context has
been lost over the years), but their main point is "we may not need
these functions", to which I say we definitely need them because
they've been a public API for over 7 years now.

Change-Id: I43a5453003e8d64d00323eb973629a96b6b386ab

resources/src/jquery/jquery.textSelection.js

index 522e95b..e7ae467 100644 (file)
                                        // Return [start, end] instead of just start
                                        startAndEnd: false
                                }, options );
-                               // FIXME: We may not need character position-based functions if we insert markers in the right places
                                break;
                        case 'setSelection':
                                options = $.extend( {
                                if ( options.end === undefined ) {
                                        options.end = options.start;
                                }
-                               // FIXME: We may not need character position-based functions if we insert markers in the right places
                                break;
                        case 'scrollToCaretPosition':
                                options = $.extend( {