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

The file status icon on tabs is not clear #100

Open
wiz0u opened this issue Sep 11, 2019 · 15 comments
Open

The file status icon on tabs is not clear #100

wiz0u opened this issue Sep 11, 2019 · 15 comments
Assignees
Milestone

Comments

@wiz0u
Copy link

wiz0u commented Sep 11, 2019

I don't understand the behavior of that little fellow icon after the file name in the tab header.
Some times it's ≡, sometimes it's an "eject down" icon that looks like the one from the "follow file" button, sometimes this "eject down" icon is red.

However I haven't found any link with the actual "follow mode" of the file, or even as an indicator that there as been new lines added since I last viewed this tab..

I don't know if it's buggy or just an explanation is required.
BTW, the 'Wiki' link on top of this github repo doesn't seem to work.

@wiz0u
Copy link
Author

wiz0u commented Sep 11, 2019

My recommendation would be:
≡ when the file is not in follow mode
red ≡ when the file is not in follow mode, is not the active pane, and new data has been detected
'eject down' when the file is in follow mode
'eject down red' when the file is in follow mode, is not the active pane, and new data has been detected

@variar variar self-assigned this Sep 14, 2019
@variar variar added this to the 2019.next milestone Sep 14, 2019
@Mte90
Copy link

Mte90 commented Sep 18, 2019

My question is more simple, what is the purpose of that option?
There is already the filesystem change monitoring option.
Maybe it is for the auto scrolling?

@variar variar modified the milestones: 2019.next, 2019.12 Sep 24, 2019
@variar
Copy link
Owner

variar commented Sep 27, 2019

It is supposed to work like this:

  • ≡ if user has interacted with the widow when tab was active (mouse clicks, hotkeys or scrolling) so there is no new data for user
  • "eject" if tab was inactive and file changed on disk and was reloaded
  • "red eject" if tab was inactive and search was set to auto-refresh and new search results appeared

@wiz0u
Copy link
Author

wiz0u commented Sep 27, 2019

Oh so 'red' is related to new search results? that seemed not intuitive to me...

Red seems like a highlight to me, meaning there is something new (new lines, not necessarily search-related).

Maybe it would work better with different kinds of highlights... For example 'blue eject' for new lines, and 'red eject' for new lines matching the auto-refresh search.

That would be more intuitive to me.

Also I was confused by the 'eject' icon because it looked very much like the 'Follow file' icon (but you changed the Follow file recently so...)

variar added a commit that referenced this issue Sep 28, 2019
@variar
Copy link
Owner

variar commented Sep 28, 2019

Made this icon to be 'blue eject' for new lines, and 'red eject' for new lines matching the auto-refresh search from build 483

@variar variar added the status: ready for testing resolved but needs testing label Sep 28, 2019
@wiz0u
Copy link
Author

wiz0u commented Sep 29, 2019

thanks, i will check that

@wiz0u
Copy link
Author

wiz0u commented Sep 30, 2019

The blue/red eject should disappear once the tab is activated, as an acknowledgement that the user has seen this notification icon.

If the file is already the active tab, there should be a way to reset this "new lines" status icon aswell, otherwise the 'eject' icon will stay forever. Maybe on any action (click, wheel, keyboard) in the window ?

@variar
Copy link
Owner

variar commented Sep 30, 2019

Eject icon should already disappear on scrolling and some mouse clicks (maybe inside the window with search results). I think the original Nick's idea behind it was to remove the icon once user scrolled to new data, but I'm not sure.

I'll try to add more user interaction events to remove it and see how it goes.

@variar variar removed the status: ready for testing resolved but needs testing label Sep 30, 2019
@wiz0u
Copy link
Author

wiz0u commented Sep 30, 2019

It disappear on scrolling, right.

Then please make it disappear also when the user switch to that tab (and the view is already at the end - because of follow mode)

@wiz0u
Copy link
Author

wiz0u commented Nov 13, 2019

As far as I can observe, that tab indicator is still not satisfying to me...

I want to be able to see at a glance in which tabs there are new lines. Tab indicator without new lines should be uncolored ≡ as to not attract the eyes.
And when I click on a tab with new lines, and go to the end (or follow-mode is active and it is already at the end) the indicator should reset immediately to uncolored ≡.
As for the current tab, if there are new lines and follow-mode is active, as soon as klogg window is focused, the indicator should also reset immediately to uncolored ≡.

@variar
Copy link
Owner

variar commented Dec 30, 2019

Made icon switch to ≡ when entering tab with changes and follow mode is enabled in 19.12.0.554.

@variar variar added the status: ready for testing resolved but needs testing label Dec 30, 2019
@wiz0u
Copy link
Author

wiz0u commented Dec 30, 2019

That's a good first step. Thanks!

@variar variar modified the milestones: 2020.01, 2020.next Dec 30, 2019
@wiz0u
Copy link
Author

wiz0u commented Dec 30, 2019

Made icon switch to ≡ when entering tab with changes and follow mode is enabled in 19.12.0.554.

Mmh does not always work, it seems.. I have a follow-mode tab that refuse to reset to ≡. Even when I force a scroll down with the mouse.

@wiz0u
Copy link
Author

wiz0u commented Dec 30, 2019

and apparently, the icon does not always turn to blue arrow when there is new lines either..
Not reliable indicator :-(

@variar variar removed the status: ready for testing resolved but needs testing label Dec 30, 2019
@wiz0u wiz0u changed the title The "follow" icon on tabs is not clear The file status icon on tabs is not clear Jan 9, 2020
@ghost
Copy link

ghost commented Apr 9, 2021

I noticed another issue which might be related, when tabs are rearranged (via DND) the indicator displays on the tab matching the index where the file was originally, not where it is now. If this is unrelated I can open a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants