Bookmark text, when "Indent with tabs" is set

[at5dapa1]

Hi,
I don't have "Indent with tabs" checked, but I have old project files which have real TABs.
On that lines, the bookmark text is not shown in the "Text" column, but into a next 2nd column, which I have to scroll for it to make it visible (see attached "bad.jpg"), and which will dissapear on any resize.

I tried quick&dirty to correct this, by adding:

Code: [Select]

LineData = strip(LineData );after each

Code: [Select]

      _BookmarkGetInfo(bm_index,BookmarkName,
                       markid,vsbmflags,bufid,
                       0,RealLineNumber,col,BeginLineROffset,LineData,
                       Filename,DocumentName);from bookmark.e.

For me the result is acceptable (see "better.jpg"). Maybe it helps somebody. I tested only in 11.0.2!
Daniel

[ScottW, VP of Dev]

Yup, good idea. I have filed a change request for this. It's small enough that it should be in v12. Does anyone see any reason not to strip the leading spaces from the Bookmark tool window?  Bueller?  Bueller?

--Scott

[hs2]

I agree that it's a good idea.

Additionally I think a clipboard history toolbar very similar to this (bookmarks) would be fine.
And a 'View' popup window (mouse-over / timer based for current entry for keyboard-only users) displaying some context lines (-/+ N lines) of the bookmark or the (condensed) clipboard contents might be also a very useful.

What do you think ?

HS2

[Ding Zhaojie]

Yup, good idea. I have filed a change request for this. It's small enough that it should be in v12. Does anyone see any reason not to strip the leading spaces from the Bookmark tool window?  Bueller?  Bueller?

--Scott

Hi, can v12 add a "Todo list" just like the "Tasks" window in Eclipse JDT? I think it is a more common way to mark important things in codes than bookmarks.

[hs2]

Good idea and a good complement to bookmarks, but IMO no real replacement for them.

@Ding:
I use this very simple, but useful (at least for me) workaround for the time being...

Code: [Select]

#include 'slick.sh'

// opt: c == current buffer, b == all buffers, p == project files, w == workspace files
_command int f2,find_2do (_str opt = 'c') name_info (','VSARG2_READ_ONLY|VSARG2_REQUIRES_EDITORCTL|VSARG2_MARK)
{
   int status = 0;
   _str search_options = make_search_options (VSSEARCHFLAG_WORD |
                                              VSSEARCHFLAG_RE |
                                              VSSEARCHFLAG_HIDDEN_TEXT |
                                              VSSEARCHFLAG_NOSAVE_TEXT |
                                              VSSEARCHFLAG_FINDHILIGHT |
                                              VSSEARCHFLAG_GO, true);
   search_options = search_options :+ '@W:PCCF';

   _str cur_word = '(TODO)|(FIXME)|(REMOVEME)|(DBG)|(HS2DO)|(DINGDO)';

   int grep_id = 9;   // -> Search<9>
   int mfff = MFFIND_THREADED | MFFIND_GLOBAL;

   clear_highlights ();
   switch ( opt )
   {
      case 'b':
         status=_mffind (cur_word, search_options, MFFIND_BUFFERS, "", mfff , false, false, '', '', true, grep_id);
         break;
      case 'p':
         status=_mffind (cur_word, search_options, MFFIND_PROJECT_FILES, "", mfff, true, false, '', '', true, grep_id);
         break;
      case 'w':
         status=_mffind (cur_word, search_options, MFFIND_WORKSPACE_FILES, "", mfff, false, true, '', '', true, grep_id);
         break;
      case 'c':
      default:
         status=_mffind (cur_word, search_options, MFFIND_BUFFER, "", mfff, false, false, '', '', true, grep_id);
   }

   cursor_data();
   return( status );
}

// (aliased) wrappers to be bound to keyboard shortcuts
_command int f2b,find_2do_buffers () name_info (','VSARG2_READ_ONLY|VSARG2_REQUIRES_EDITORCTL|VSARG2_MARK)
{
   return(find_2do ('b'));
}
_command int f2p,find_2do_project () name_info (','VSARG2_READ_ONLY|VSARG2_REQUIRES_EDITORCTL|VSARG2_MARK)
{
   return(find_2do ('p'));
}
_command int f2w,find_2do_wkspace () name_info (','VSARG2_READ_ONLY|VSARG2_REQUIRES_EDITORCTL|VSARG2_MARK)
{
   return(find_2do ('w'));
}

HS2

[at5dapa1]

If you're going to update "Bookmarks", then it's not enough only to strip the string. You must also replace the tabs inside the string, with spaces. Else again we'll have something on the next column.