search

adixon / ca.civicrm.logviewer

CiviCRM Log Viewer
Other
9 stars 11 forks source link

Add navigation to go prev/next log entry #6

Closed sluc23 closed 6 years ago

sluc23 commented 6 years ago

Add navigation to go prev/next log entry

screenshot_2018-09-04 log entry view dev hosting lspiegel server-dev ixiam lan

adixon commented 6 years ago

Very nice!

sluc23 commented 6 years ago

thanks @adixon ! I haven't changed info.xml, so if you think it's ok would be good to have a new release/tag of this extension

adixon commented 6 years ago

Yeah, I've got it in testing on my sites, I'll generate a release in a couple of days if no one hits an issue.

The only thing I've noticed so far is perhaps to not show an active link if there is no next/prev (e.g. if you're on the last entry, you still get an active "next" link.) Just a tiny ux thing ...

sluc23 commented 6 years ago

yeap, I thought the same about hiding the links. Fixed in PR #7