Fixed issues in history view #4427
Conversation
There was a problem hiding this comment.
looks good to me. Seems to work too.
If you want to get this into NetBeans 15, you would have to put the commit on top of the delivery branch and then force push to this PR again. Master is for 16 already.
btw: I found another small issue while testing: if you select "Load All" it should probably remove the "Load next 30 days" item, since there shouldn't be anything more to load.
neilcsmith-net
added
the
do not merge
|
To help release tracking, have marked with NB15 and do-not-merge as and until the change in branch is made as suggested by @mbien |
…next 30 days" in history view
troizet
force-pushed
the
fix_history_view
branch
from
649c92f
to
779f6a7
Compare
|
Implemented the suggested changes to the branch. @mbien, @neilcsmith-net please check if I did it right. |
|
@mbien, fixing the "Load All" issue, I think, is better done in another PR. |
looks good, thanks for rebasing on delivery. |
neilcsmith-net
removed
the
do not merge
Hello! This PR fixes issues #4367 and #4394.