Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Console view toolbar not shown after scrolling on v23.4.0 #12220

Closed
asavchkov opened this issue Nov 6, 2023 · 2 comments · Fixed by #12300
Closed

Console view toolbar not shown after scrolling on v23.4.0 #12220

asavchkov opened this issue Nov 6, 2023 · 2 comments · Fixed by #12300

Comments

@asavchkov
Copy link

Issue Type
  • Bug Report
Summary

In older versions of go-server up to 23.3.0 an toolbar was always available at the top of the console view regardless of the current scrolling position, i.e. no matter at which position of the console log I was (top, middle, bottom) this toolbar was present as a dynamic overlay panel. This is the toolbar I am talking about:
v23 3-panel
I have always found it very convenient and used it a lot to toggle the automatic log update, timestamps and the color theme (at times some text was hard to read with its current colors).

In v24.4.0 this toolbar no longer follows the current view but rather scrolls normally with the rest of the log content and immediately goes out of the view if I move a couple of lines down in the log.

Could you please put this nice toolbar back such that it is always available at the top of the view when scrolling console logs?

Basic environment details
  • Go Version: 23.4.0 (17731-4deb96d823b921419680560be080588e900d406e).
  • JAVA Version: 17.0.9
  • OS: Linux 5.15.0-88-generic
  • Browser vendor and version (if relevant): Mozilla Firefox 119.0
@chadlwilson
Copy link
Member

There was no intention to change this, but it might have been a side effect of something else. There is a lot of old untested stuff on these views, unfortunately.

It might have been an accidental victim of a view framework upgrade like #11817, or some other CSS cleanup like 7274e1f (having no idea how it worked before).

@chadlwilson chadlwilson added the bug label Nov 6, 2023
@chadlwilson chadlwilson added this to the Release 23.5.0 milestone Nov 6, 2023
@chadlwilson
Copy link
Member

Ahh, just keeping a note for myself - it's likely that

$.fn.pinOnScroll = function (opts) {
this.each(function () {
var elem = $(this);
var throttledFixElement = _.throttle(fixElement, 100);
var throttledUnFixElement = _.throttle(unFixElement, 100);
$(window).on('scroll.pinOnScroll', function () {
maybeCallUsingRequestAnimationFrame(function () {
throttledFixElement(elem, opts);
});
});
$(window).on('resize.pinOnScroll', function () {
maybeCallUsingRequestAnimationFrame(function () {
throttledUnFixElement(elem, opts);
throttledFixElement(elem, opts);
});
});
$(window).on('resetPinOnScroll', function (e, eParams) {
if (eParams.calcRequiredScroll) {
opts.requiredScroll = eParams.calcRequiredScroll();
}
throttledUnFixElement(elem, opts);
throttledFixElement(elem, opts);
});
});
return this;
};
}(jQuery));
is not working as expected with the newer jquery versions.

@chadlwilson chadlwilson self-assigned this Nov 6, 2023
@chadlwilson chadlwilson changed the title Console view toolbar not shown in v23.4.0 Console view toolbar not shown after scrolling on v23.4.0 Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants