Menu

#253 Do not refresh data grid on selection (or mark as read)

1.6.8
Fixed
nobody
None
Medium
Defect
2010-08-01
2010-03-17
Anonymous
No

Originally created by: prodevst...@gmail.com
Originally owned by: tortoisesvn

What steps will reproduce the problem?
1. After windows start open CommitMonitor or Scan for new commit
2. Assuming you have new commit and comments click on a unread comment/[r3]. your selection is cleared/data grid refreshed and you have no focus on
the actual revision/commit comment

What is the expected output? What do you see instead?
It should highlight the selected commit comment and mark it as read. But
when it marks it as read it refreshes the entire data grid. It should not
need to refresh entire data grid to mark single item as read.

What version of the product are you using? On what operating system?
CommitMonitor 1.6.7.509 on Windows XP (SP3)

Please provide any additional information below.
It makes it difficult to use the keyboard to go through the commit comments
one by one as the focus goes back to top of the data grid again. So you
have to scroll down again from top.

Discussion

  • Anonymous

    Anonymous - 2010-08-01

    Originally posted by: tortoisesvn

    this only happens if you view the commits while new commits are added (i.e., a check is currently in progress that detected new commits and adds them one by one as it fetches them).
    Just marking an entry as read doesn't refresh the grid, only if new commits are added it gets refreshed.

     
  • Anonymous

    Anonymous - 2010-08-01

    Originally posted by: tortoisesvn

    I'll add some code to restore the selection mark after such a refresh. But it won't get rid of this completely: depending on how many new entries were added between the refreshes, it might not get restored correctly.

    But since that doesn't happen that often, I guess that will do.

    Labels: Milestone-1.6.8
    Owner: tortoisesvn
    Status: Started

     
  • Anonymous

    Anonymous - 2010-08-01

    Originally posted by: tortoisesvn

    This issue was closed by revision [r528].

    Status: Fixed

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.