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

[UI] - Settings > Labels - sort by description has no effect #13950

Closed
karel-rehor opened this issue May 17, 2019 · 2 comments · Fixed by #15515
Closed

[UI] - Settings > Labels - sort by description has no effect #13950

karel-rehor opened this issue May 17, 2019 · 2 comments · Fixed by #15515
Assignees
Labels
area/ui cypress kind/bug team/ui testing ui/design-polish Something in the UI needs a small tweak (not a whole redesign)

Comments

@karel-rehor
Copy link
Contributor

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

  1. Open Settings > Labels
  2. Create 5 or more labels including various descriptions
  3. Sort by name ascending - OK
  4. Sort by name descending - OK
  5. Sort by description ascending - looks like the result is sort by name
  6. Sort by description descending - no change

Expected behavior:

Expected that sorting by description should work by alpha, whether ascending or descending, based on values in the description field.

Actual behavior:

First description sort followed order of strings in name field. Attempt to sort in opposite direction resulted in no change to the sort order.

Environment info:

  • System info: Run uname -srm and copy the output here Linux x86_64
  • InfluxDB version: Run influxd version and copy the output here - local build for creating cypress tests. Last commit from master d31be2a
  • Other relevant environment details: Container runtime, disk info, etc
    Browser: Chrome 73
    Config:
    Copy any non-default config values here or attach the full config as a gist or file.

Boot command

./bin/linux/influxd --store=memory --e2e-testing=true --log-level debug 2>&1 > ~/var/log/influxd.log &

Screenshots
LabelsSortByName01
LabelsSortByName02
LabelsSortByDesc01
LabelsSortByDesc02

@karel-rehor karel-rehor added kind/bug ui/design-polish Something in the UI needs a small tweak (not a whole redesign) area/ui cypress labels May 17, 2019
@russorat russorat modified the milestones: diamond, Sprint 5 - Emerald Jul 22, 2019
@russorat russorat removed this from the Sprint 6 - Fluorite milestone Aug 26, 2019
@karel-rehor
Copy link
Contributor Author

In latest nightly build (commit=4f7e147e2 build_date=2019-10-03T07:16:23Z) - this issue is still present

ts=2019-10-03T08:00:46.100490Z lvl=info msg="Welcome to InfluxDB" log_id=0IGA0S50000 version=nightly commit=4f7e147e2 build_date=2019-10-03T07:16:23Z

Screencast
SortLabelsByDescription01

Acceptance Test Note

I'm currently adding tests for the labels tab to the acceptance test suite. I would like to add the sort labels by description use case.

@karel-rehor
Copy link
Contributor Author

Related Issues

#15397
#15301
#15247
#14923

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui cypress kind/bug team/ui testing ui/design-polish Something in the UI needs a small tweak (not a whole redesign)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants