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.

Sign up for GitHub

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

Jump to bottom

Dashboard cell contents disappear after canceled edit, require page refresh to view #15239

Closed
glinton opened this issue Sep 23, 2019 · 2 comments · Fixed by #15559
Closed

Dashboard cell contents disappear after canceled edit, require page refresh to view #15239

glinton opened this issue Sep 23, 2019 · 2 comments · Fixed by #15559
Assignees
@hoorayimhelping
Labels
kind/bug testing

Comments

@glinton
Copy link
Contributor

glinton commented Sep 23, 2019

Steps to reproduce:

  1. Open dashboard in cloud2
  2. Select "Configure" from blue pencil icon in dashboard cell (graph, etc...)
  3. Choose the "X" in the top right (next to green checkmark)
  4. Observe missing cell contents

Expected behavior:
Cell contents, same behavior as selecting the green check in cell configuration mode.

Actual behavior:
Contents disappear until a page refresh is executed.

Environment info:

  • System info: Linux 5.0.0-27-generic x86_64
  • InfluxDB version: ?? cloud2.influxdata.com
  • Other relevant environment details: Chrome Version 77.0.3865.75 (Official Build) (64-bit)
@desa desa added the kind/bug label Oct 7, 2019
@desa
Copy link
Contributor

desa commented Oct 14, 2019

@russorat is currently unable to reliably reproduce.

@russorat
Copy link
Contributor

russorat commented Oct 21, 2019 •
edited
Loading

I am able to repro. hitting the X to close any cell causes this to happen.

Kapture 2019-10-21 at 11 47 39

@hoorayimhelping hoorayimhelping self-assigned this Oct 22, 2019
@hoorayimhelping hoorayimhelping mentioned this issue Oct 24, 2019
Merged
4 tasks
@desa desa added the testing label Oct 24, 2019
hoorayimhelping added a commit that referenced this issue Oct 25, 2019
@hoorayimhelping
test(ui): add e2e regression test for issue #15239
625483f
@hoorayimhelping hoorayimhelping mentioned this issue Oct 25, 2019
Merged
3 tasks
hoorayimhelping added a commit that referenced this issue Oct 25, 2019
@hoorayimhelping
test(ui): add e2e regression test for issue #15239
038d6bf
hoorayimhelping added a commit that referenced this issue Oct 25, 2019
@hoorayimhelping
test(ui): add e2e regression test for issue #15239
89b7547
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoorayimhelping hoorayimhelping

Labels
kind/bug testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(ui): exiting out of a configure cell leaves cell content intact influxdata/influxdb
4 participants
@hoorayimhelping @desa @glinton @russorat