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

Dragging and reordering rows can break queries #19

Closed
danepowell opened this issue Jul 27, 2018 · 7 comments
Closed

Dragging and reordering rows can break queries #19

danepowell opened this issue Jul 27, 2018 · 7 comments
Labels

Comments

@danepowell
Copy link

Note that this is a bug report for v1.1. This problem appears to be solved in v2, but I wanted to post an issue in case any other users of v1.1 ran across it.

If you try to reorder rows on the Bundles sheet by dragging them around, this can break the rest of the spreadsheet in unexpected ways.

You can recreate this by simply dragging a row from midway down the bundles sheet up into the first row of the sheet (actually row 2 since there's a header).

This will cause the bundles query to "helpfully" automatically adjust itself to only use row 3 and onward.

@danepowell
Copy link
Author

danepowell commented Jul 27, 2018

Actually, there's another place where dragging rows breaks things, even in v2: if you drag rows around on the fields tab, all of the conditional formatting goes haywire. Maybe it's best just to advise people not to do that...

@TravisCarden
Copy link
Contributor

Thanks, @danepowell. The first problem of broken queries was a known issue in v1.x which has indeed been solved in v2.0.0. Unfortunately, I have no way to get the fix out to users of v1.1. Since there's no update delivery mechanism for Google Sheets, each version does become effectively unmaintainable once released. (If you have thoughts on that problem, I'm all ears!)

As to the conditional formatting going haywire when reordering rows on the "Fields" tab, I'm aware of a problem with zebra stripes breaking in that scenario. Do you see other formatting breaking?

@danepowell
Copy link
Author

The two major problems were the zebra stripes (aka conditional formatting), and the hidden "type" column. For some reason when you drag rows around, the "type" column becomes populated with literal text values, and this breaks the auto-population function. I've confirmed both of these happen in v2

@TravisCarden
Copy link
Contributor

I may be able to make the zebra stripes code more resilient. I'm not sure the issue with the "Type" column is surmountable, though. I wonder if the column is worth the trouble. What do you think, @danepowell?

@TravisCarden TravisCarden changed the title Dragging rows around on Bundle sheet can break queries (v1.1) Dragging and reordering rows can break queries Aug 1, 2018
@danepowell
Copy link
Author

I'm actually not sure what that column does, since it's normally hidden right? I'm okay either way, if we decide not to fix it then maybe just document "avoid dragging rows".

@TravisCarden
Copy link
Contributor

The column doesn't actually "do" anything in the sense of contributing to validation, Behat tests, or diagrams. It's purely informational. And it's brittle in more than one way. 🤔 I think I'm going to get rid of it. I'm not willing to make a concession about such a basic behavior as dragging rows unless absolutely unavoidable.

@TravisCarden TravisCarden added the bug Something isn't working label Aug 3, 2018
@TravisCarden
Copy link
Contributor

@danepowell, I've decided to remove the zebra stripes until they can be implemented more robustly. As before, I'm not willing to recommend that people give up common spreadsheet features and behaviors to compensate for the quirks of a nicety. I'm closing this issue "fixed". Feel free to reopen it or create a new one if you discover anything else. Thanks!

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

No branches or pull requests

2 participants