Webstaff grid could be smarter in IDL mode

Bug #1683385 reported by Mike Rylander on 2017-04-17
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Medium
Unassigned
3.2
Undecided
Unassigned

Bug Description

The grid we've built for general use in the web staff client has a mode where it auto-configures columns based on a named IDL class. This is great!

However, if you need to add specialized columns (such as for linking barcodes or formatting timestamps) then you get extra "raw" versions of those columns in the column picker. It would be a kindness if the grid would notice that a path has been used explicitly and skip the column addition.

The wildcard syntax for column addition knows how to do this, so hopefully that code can be cribbed.

Andrea Neiman (aneiman) on 2017-05-10
tags: added: webstaffclient
Kathy Lussier (klussier) on 2017-10-31
tags: added: webstaffcolumns
Kathy Lussier (klussier) wrote :

As I look at bug 1669866, bug 1722387 and bug 1705133, it might be a good idea to consider whether it is indeed a good thing to configure columns based on a named IDL class. Is this the reasons why we are getting very long column pickers in some interfaces that include data that will rarely be used? Would it be better to explicitly add all columns rather than adding an entire class at one time?

tags: added: needsdiscussion
Tiffany Little (tslittle) wrote :

I'd like to revive discussion on this bug because I'm interested in what others think, and based on Kathy's comment maybe this is the best place. Explicitly adding columns is what I've done here: https://bugs.launchpad.net/evergreen/+bug/1724965 but I don't know if that's the best path forward.

Having the columns info autogenerated is nice and clean coding-wise, but it just dumps in all the fields in that class, some of which aren't useful for that particular screen. And if you need more than one class (like in the Checkout screen, I think?) then you get uber-long column picker options which again, some of them aren't pertinent. It also doesn't let you do things like having a user barcode/username vs. the user's ID, which is friendlier.

I'm interested in others' thoughts if there's a better way to resolve all that.

Bill Erickson (berick) wrote :

Original issue confirmed.

I also note that grid column configurations do not apply correctly to columns with auto-fields since the auto fields are generated after the column configuration is applied.

As to the auto-fields vs. manual fields debate, I say any time manual fields are better, use them. In some cases (especially admin UI's) auto fields save us a lot of typing and generally work well, so retaining and improving the functionality would be best.

Note these issues are resolved in the Angular grid.

Grabbing bug to address the duplication issue.

Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
importance: Wishlist → Medium
status: New → In Progress
Bill Erickson (berick) wrote :
Changed in evergreen:
milestone: none → 3.3.1
assignee: Bill Erickson (berick) → nobody
status: In Progress → Confirmed
tags: added: pullrequest
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers