Project

General

Profile

Actions

action #51719

closed

"New Test Suite " button in admin/test_suites can not be used with filter

Added by asmorodskyi almost 5 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2019-05-21
Due date:
% Done:

0%

Estimated time:

Description

  1. Define any filter in admin/test_suites
  2. Press "New Test Suite" button ==> nothing happens

Expected : empty line for new test suite should appear ( with filter reset in case there is no other way )

Note : if you will remove filter after button pressed you will get empty line for new test suite so it just simple filtered out


Related issues 2 (1 open1 closed)

Related to openQA Project - action #53402: [beginner] Not possible to search for test suites (again?) using regex, e.g. `^gnome$`Workable2019-06-21

Actions
Has duplicate openQA Project - action #52385: Button 'New test suite' does not work after using table search.Rejected2019-05-31

Actions
Actions #1

Updated by mkittler almost 5 years ago

  • Category set to 124

Yes, it simply gets filtered out. Maybe I can prevent that somehow (1) or at least disable the button as long as filter are active (2). If I go for (1) the result might of course be filtered out when submitting. In case of (2) a hint why the button is disabled would likely be useful.

Actions #2

Updated by asmorodskyi almost 5 years ago

mkittler wrote:

Yes, it simply gets filtered out. Maybe I can prevent that somehow (1) or at least disable the button as long as filter are active (2). If I go for (1) the result might of course be filtered out when submitting. In case of (2) a hint why the button is disabled would likely be useful.

regarding filter out on submit - something similar happens currently anyway : due to pagination you are in high risk that test suite which you just created got not into first page so from user perspective it will just disappear. Not sure how it could be handled better so haven't post a ticket about that . But just mention this here to clarify that if you choose (1) you will not create some regression because this behavior exists already

Actions #3

Updated by mkittler almost 5 years ago

Yes, that's true. However, the problem that rows "disappear" after submitting due to their new position in the sort order could be solved elegantly by simply selecting the new page of the row.

Filtering is more problematic. Maybe we could simply adjust the filter in cases rows should not disappear? So creating a new test suite would clear the filter and updating a test suite would adjust the filter to include the new name as well. Than the adjusted filtering would be transparent to the user and there are also no internal hacks required.

Actions #4

Updated by asmorodskyi almost 5 years ago

mkittler wrote:

Yes, that's true. However, the problem that rows "disappear" after submitting due to their new position in the sort order could be solved elegantly by simply selecting the new page of the row.

you mean autoselecting ? yeah I like this idea ! because currently it is really confusing .

Filtering is more problematic. Maybe we could simply adjust the filter in cases rows should not disappear? So creating a new test suite would clear the filter and updating a test suite would adjust the filter to include the new name as well. Than the adjusted filtering would be transparent to the user and there are also no internal hacks required.

I would not use this way because I am afraid of plenty different scenario where such adjustment will create more problems that it solves. I would go for simple filter reset in case "New Test Suite" button was pressed. Easy to implement and pretty transparent to the user :)

Actions #5

Updated by mkittler almost 5 years ago

  • Has duplicate action #52385: Button 'New test suite' does not work after using table search. added
Actions #6

Updated by mkittler almost 5 years ago

  • Assignee set to mkittler
  • Target version set to Ready
Actions #7

Updated by mkittler almost 5 years ago

  • Status changed from New to In Progress
  • Target version changed from Ready to Current Sprint

PR for "autoselecting" of updated rows and resetting the filter before adding a new/empty row: https://github.com/os-autoinst/openQA/pull/2089

Actions #8

Updated by okurz over 4 years ago

  • Status changed from In Progress to Resolved

PR merged, live on https://openqa.opensuse.org/admin/test_suites and working as expected with proper focussing as well as filter resetting

Actions #9

Updated by okurz over 4 years ago

  • Related to action #53402: [beginner] Not possible to search for test suites (again?) using regex, e.g. `^gnome$` added
Actions

Also available in: Atom PDF