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
[dev.icinga.com #13115] actiontable should not clear active row in case there is no newer one #2615
Comments
Updated by elippmann on 2016-11-09 16:48:22 +00:00 Hi Tom, When does this happen? Best regards, |
Updated by elippmann on 2016-11-09 17:08:55 +00:00
Is this something we've introduced after 2.3.4? |
Updated by tgelf on 2016-11-09 17:13:47 +00:00 Don't think so, I got angry about this many times before :D But in "normal live" it is just confusing. Today however I wanted to create a new feature and haven't been able to do so because of this issue, so I thought time was right to finally tackle this. |
Updated by tgelf on 2016-11-09 18:11:04 +00:00
Some more detailsrefresh() basically does clear(); and selectUrl(); - I'd prefer to see something like if (newRow = findNewRow()) { selectRow(newRow); } - with clear(); being called in selectRow. Please note that this might involve some logic dealing with other related tables, read "dashboard". That one currently works find as of clear & select, it wouldn't anymore when fixing this. Now lets move on with the problem. In selectUrl() we finish here:
This is a workaround for my problem, but it works out for very simple tab constructs on the right side, with all of them having the same parameters. This doesn't work out for the Director, as it often involves different controllers on the right side while keeping the very same table on the left. You might navigate to a host, show it's services, click a service - the row gets deselected, while your are still in the same context. Or go to deployments, choose one, show it's config. Which one to click next? You have no idea which row you are currently working with. Fixing this issue would also allow me to preselect a specific row. Click a specific config file belonging to a deployed config. It will be rendered in the same container and provide a back link. What I'd like to do is modifying that back-link in a way it would pre-select the row you formerly clicked, making it easier to figure out where you started your investigation. So, basically that's it where this issue started - because I'm currently unable to implement this. But regardless of this feature, the current behavior of toggling the selected row on-off-on-off when navigating through the tabs in the right container is confusing and should be fixed. |
Updated by tgelf on 2016-11-09 19:17:53 +00:00 I pushed feature/actiontable-row-handling-13115 - tries to address all mentioned issues. |
Updated by tgelf on 2016-11-09 19:39:38 +00:00
|
Updated by tgelf on 2016-11-16 09:35:07 +00:00
|
Updated by elippmann on 2016-12-09 12:50:04 +00:00
Applied in changeset aa559c4. |
This issue has been migrated from Redmine: https://dev.icinga.com/issues/13115
Created by tgelf on 2016-11-09 16:41:44 +00:00
Assignee: (none)
Status: Resolved (closed on 2016-12-09 12:50:04 +00:00)
Target Version: 2.4.0
Last Update: 2016-12-09 12:50:04 +00:00 (in Redmine)
Currently it removes all class="active" and then tries to figure out whether there is a new candidate that should be elected active. Blind guess: this is probably only a quick hack to solve issues on the dashboard, as it formerly didn't work like this (long time ago). We should IMO get this corrected.
This is confusing when:
Hint: when you try to work around this, issue #13113 makes the whole thing even worse.
Changesets
2016-11-09 19:16:00 +00:00 by tgelf d2710cf
2016-11-10 00:40:42 +00:00 by tgelf 2d58c85
2016-11-16 09:38:44 +00:00 by tgelf 0f58776
2016-12-09 12:49:27 +00:00 by elippmann aa559c4
Relations:
The text was updated successfully, but these errors were encountered: