theia-cpp-extensions icon indicating copy to clipboard operation
theia-cpp-extensions copied to clipboard

Show parameters of current display

Open colin-grant-work opened this issue 4 years ago • 1 comments

Context: currently, the only indication of the parameters with which the current display was fetched is the actual values of the configuration inputs (address, offset, length). If the user changes those values and either doesn't try to fetch memory or tries and fails, the display is not updated, and we lose the connection between the inputs and the current state of the display. It would be desirable to display the parameters used for the current state of the table apart from the inputs so that the user can always tell what context the data displayed comes from.

Maybe an alternative would be to add some information to the table display indicating the parameters with which the data displayed was fetched. That would at least address the potential for confusion.

I personally like this idea. Since it might require a little bit of UI re-design maybe it's better left for a separate PR if that's OK with you.

Originally posted by @federicobozzini in https://github.com/eclipse-theia/theia-cpp-extensions/issues/132#issuecomment-941172038

colin-grant-work avatar Oct 12 '21 16:10 colin-grant-work

@colin-grant-work another idea for this issue.

How would you feel about moving the search area outside of the tab?

At the moment a user needs to click on the top-right button to create a new memory tab, then perform a search. Maybe we could just put the search parameters at the top of the panel and the user would only need to perform a search to get a new tab created. If the search is successful and the tab is correctly created we might show there the parameters used for the search in the new tab, which would be only updatable when navigating the memory using the Load more buttons.

federicobozzini avatar Oct 13 '21 12:10 federicobozzini