vicoapp / vico

Mac Programmers Text Editor
http://www.vicoapp.com/
657 stars 89 forks source link

Forward Jump button doesn't re-enable itself when it's supposed to. #67

Closed farmdawgnation closed 10 years ago

Shadowfiend commented 10 years ago

Well, that's a detailed bug report :p

In particular, the issue here is that once you hit the back button for the jump list, or type C-o to go back in the jump list, the forward button doesn't enable. You can still use Tab to go forward in the jump list, but you cannot use the button to do it.

Shadowfiend commented 10 years ago

I'll try and have a look at this once I finish cleaning up the XCode build process and updating the Makefile.

Shadowfiend commented 10 years ago

This is now fixed in the jumping-jackalopes branch. Not opening a pull request yet as I ran across a comment in the source about a better way to handle the enabledness of those buttons that I want to try and implement first.

Shadowfiend commented 10 years ago

Went ahead and merged this via 24779dace027b4a8ea634265c1c7ccb698b7905e. The other approach that was mentioned didn't seem to have a particularly large upside compared to the current approach, so I'm leaving it alone for now.