Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Make the parameter names visible in filters #411

Closed
metas-rc opened this issue Feb 23, 2017 · 19 comments
Closed

Make the parameter names visible in filters #411

metas-rc opened this issue Feb 23, 2017 · 19 comments

Comments

@metas-rc
Copy link
Member

Type of issue

  • Feature request (enhancement)

Current behavior

  • When a filter is opened, the longer names of parameters are nod completely displayed

Expected behavior

  • The whole names should be seen.

Steps to reproduce

Please check the pic below:

image

@teosarca teosarca added this to the 2017-09 milestone Feb 23, 2017
@damianprzygodzki damianprzygodzki self-assigned this Feb 24, 2017
@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented Feb 24, 2017

The filters have type secondary from the default, what means that the label and the input are sized with ratio 1:3.

We cannot allow to auto-size labels in secondary input due to layout assumptions.

Also we cannot fix the width of label to some length, because there is too much cases to fill them all.

What we can do here to display more is to provide from API type of widget like primary and then do not use fallback to secondary and achieve the 100% width label above the input. What it may cause is doubled height of filters dropdown.

@metas-mk metas-mk modified the milestones: 2017-12, 2017-09 Mar 6, 2017
@cadavre cadavre modified the milestones: 2017-13, 2017-12 Mar 21, 2017
@metas-mk metas-mk modified the milestones: 2017-16, 2017-13 Apr 5, 2017
@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented Apr 20, 2017

@metas-rc Do we have some new decision made here? Or can we close issue?

@teosarca
Copy link
Member

Check with "primary" layout and it looks like this (see screenshot).

  • for big filters, if u have to scroll, when trying to scroll the filter panel is closed => NOK
  • the checkboxes does not look ok

@metas-mk wdyt?

image

@metas-mk
Copy link
Member

Let' try to get rid of the most big filters. We already started in material receipt to only have specialty filters where most of parms are already set as constants. If we manage to do that, I believe that we can go the path of primary layout BUT not for flags (looks horrible). WDYT?

@damianprzygodzki
Copy link
Contributor

Than please just change it on API side to secondary for flags. So it is no more frontend issue, can we close it?

@teosarca
Copy link
Member

Hi @damianprzygodzki
I don't think so, because:

  • the "type:primary|secondary" is considered for all filter parameters, and not for each one individually (see screenshot)
  • more, personally, cannot imagine a place where a checkbox with type:primary which is actually layouted on 2 lines looks good => i think we might want to always layout the checkbox and label on same row.

image

@damianprzygodzki
Copy link
Contributor

@teosarca so change the type property to be inside the parameters - it is resposible for single widget.

we cannot differentiate and display one widget different than others. We wanted to make generic widgets and thats the way they are working, property 'type' is defining their design. It relay on API side.

@damianprzygodzki
Copy link
Contributor

I think that frontend gives you now full ability to manipulate widgets layout in filters, so i am adding integrated flag instead of mf turn.

@teosarca
Copy link
Member

pls fix the checkboxes layout.

@damianprzygodzki
Copy link
Contributor

It is fixed, also with buttons, labels and switches. Isnt it working?

@teosarca
Copy link
Member

didn't check it... thought it wasn't since i didn't see any commits since last talked.

moving it to testing then...

@damianprzygodzki
Copy link
Contributor

Thank you

metas-ts added a commit to metasfresh/metasfresh that referenced this issue Apr 28, 2017
[#332](metasfresh/metasfresh-webui-api-legacy#332) Empties window without pre-selection possible
[#695](metasfresh/metasfresh-webui-frontend-legacy#695) Lager photo preview screen/ modal overlay
[#328](metasfresh/metasfresh-webui-api-legacy#328) Manufacturing receipt mandatory Field not filled
[#1388](#1388) HU from Cost Collector is locked and cannot be selected anymore after the CC was reversed
[#683](metasfresh/metasfresh-webui-frontend-legacy#683) Tooltip on first level breadcrumb
[#682](metasfresh/metasfresh-webui-frontend-legacy#682) Sidelist opening shall automatically have the first line selected
[#322](metasfresh/metasfresh-webui-api-legacy#322) Manufacturing Issue/Receipt: Fix Qty and QtyPlan columns
[#676](metasfresh/metasfresh-webui-frontend-legacy#676) Process result: implement openDocument as modal
[#1358](#1358) Drop de.metas.order.callout.CheckDouble
[#411](metasfresh/metasfresh-webui-frontend-legacy#411) Make the parameter names visible in filters
@metas-lc metas-lc self-assigned this Apr 28, 2017
@metas-lc
Copy link

IT
to test it after this task is ready: metasfresh/metasfresh-webui-api-legacy#334

@metas-lc
Copy link

metas-lc commented May 3, 2017

IT
mainly it's fine, but the checkboxes are a bit too close and you can't see the whole text
filter

we would need a distance ...something like you can see in sales order for example (window/143/1005135)
checkbox_order

@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented May 8, 2017

We are back again in the beginning. That's my solution.

The filters have type secondary from the default, what means that the label and the input are sized with ratio 1:3.

We cannot allow to auto-size labels in secondary input due to layout assumptions.

Also we cannot fix the width of label to some length, because there is too much cases to fill them all.

Beside that, we have already implemented tooltip for labels, because the width of labels relay on width of your browser width.
screen shot 2017-05-08 at 10 30 15

The case which you provided:

we would need a distance ...something like you can see in sales order for example (window/143/1005135)

has the same behavior in filters and in window/143, but the point is that you have much more place for widget.

What i can propose is to add additional styling type (side by side 'primary' and 'secondary') to allow change this ratio, or align the checkbox to the right, to keep consistency for all of that widget type.

@cadavre
Copy link
Contributor

cadavre commented May 8, 2017

Due to our auto-generator of UI, we always keep label to input proportion 1:3 in secondary type inputs (this means also always for checkbox, switch and button widgets). Filter window uses same UI generator as document window – the difference is document window is simply wider – it gives it more space for label. Solution might also be making filter container wider but it won't solve other cases with ellipsis for checkboxes (like users with lower screen resolutions).

@metas-lc keep in mind we cannot adjust widgets per window because we use one, single and unified UI generator – making change in one place makes the changes in all of them. Please propose your solution in this perspective (look at @damianprzygodzki proposal above).

@teosarca
Copy link
Member

teosarca commented May 9, 2017

as discussed, the backend shall provide (layout) type:"different type" (TBD) .

@teosarca
Copy link
Member

Update: introduced "primaryLongLabels" layout type.
Will be available on next rollout.

image

@metas-lc
Copy link

IT
text in filters are visible. Checkboxes are far enough and on the same line. OK
other filters which are not checkboxes, have the text on separate rows OK

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

No branches or pull requests

7 participants