speedy32129 / time_logger

Redmine Time logger plugin. More than likely there is only going to be one more public release. New version has new time logging with the option of multiple loggers running at the same time, fixes for excessive loggng, and a whole new look.
Other
59 stars 43 forks source link

Fixes for the context menu. #9

Closed tvdeyen closed 8 years ago

tvdeyen commented 9 years ago

The links for the time logger actions in the context menu of an issue are incorrect. This fixes them.

tvdeyen commented 9 years ago

I added commits that also fix the css classes and fixes for the replacement of the time logger menu. Latter only works after #10 was merged.

speedy32129 commented 9 years ago

tvdeyen can you send me a screen shot of what you are seeing with the wrong links? I just want to verify that. And thanks for all your help. And if you don't mind I'd like help with adding more transitions when a person clocks off or stops a timer. So there will be start and stop transitions. So when a person either starts a timer on a new or open it transitions to "in progress" then when a user stops it returns to open unless a user changes the status.

What do you all think? I think seeing exactly what someone is working on is better.

tvdeyen commented 9 years ago

Sorry, I can’t send screenshots, because I already updated my Redmine installation with the fix.

Just open the context menu and see for your self, sorry.

The link_to syntax used is old Rails 2 based syntax, my fix uses the new Rails 3 based link_to syntax.

There is not much to verify ;)