-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Scroll to created column when extracting or combining columns in chill mode #42650
Conversation
Codenotify: Notifying subscribers in CODENOTIFY files for diff 44cb385...ffba4e0.
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@romeovs I think we need to toggle the flag off to not scroll on new columns added by other methods (e.g. showing the column that was previously hidden). Could you toggle the flag after scrolling? Maybe in componentDidUpdate
or something like that
dd1dd87
to
4e6336e
Compare
@ranquild please have another look. |
…l mode (#42650) * Set scrollToLastColumn ui control from column extract action * Scroll to last column when the uiControl is set * Scroll to last column when combining a column * Scroll to last column when combining a column via the + shortcut * Scroll to last column when extracting a column via the + shortcut * Remove all references to settingsSyncOptions * Check that the column is added at the end * Disable scrollToLastColumn once used * Add test that verifies scrollToLastColumn ui control has been reset
@romeovs this was the first commit after the v50 release branch was cut, so if you mean to backport to 49, you'll have to do that using this command: https://www.notion.so/metabase/Branching-Strategy-and-Backports-6eb577d5f61142aa960a626d6bbdfeb3?pvs=4#29927310b3b444efbb39edff6d62c549 |
…l mode (#42650) (#42832) * Set scrollToLastColumn ui control from column extract action * Scroll to last column when the uiControl is set * Scroll to last column when combining a column * Scroll to last column when combining a column via the + shortcut * Scroll to last column when extracting a column via the + shortcut * Remove all references to settingsSyncOptions * Check that the column is added at the end * Disable scrollToLastColumn once used * Add test that verifies scrollToLastColumn ui control has been reset Co-authored-by: Romeo Van Snick <romeo@romeovansnick.be>
…l mode (#42650) * Set scrollToLastColumn ui control from column extract action * Scroll to last column when the uiControl is set * Scroll to last column when combining a column * Scroll to last column when combining a column via the + shortcut * Scroll to last column when extracting a column via the + shortcut * Remove all references to settingsSyncOptions * Check that the column is added at the end * Disable scrollToLastColumn once used * Add test that verifies scrollToLastColumn ui control has been reset
Closes #42674
Description
Describe the overall approach and the problem being solved.
How to verify
Describe the steps to verify that the changes are working as expected.
a. Click the Email column header -> Click
Extract domain, host
->Host
b. Click the Email column header -> Click
Combine column
->Done
c. Click the
Add column
button in the table header ->Extract part of this column
->Email
->Host
d. Click the
Add column
button in the table header ->Combine columns
-> select some columns ->Done