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

Cut-Off Header names #1051

Closed
metas-mk opened this issue Jul 18, 2017 · 8 comments
Closed

Cut-Off Header names #1051

metas-mk opened this issue Jul 18, 2017 · 8 comments

Comments

@metas-mk
Copy link
Member

Type of issue

Feature request

Current behavior

Currently, it happens in header names that the columns are sometimes too narrow so be shown completely and then are cut off, which looks untidy.

Expected behavior

In case a header label does not fit properly, show '...' instead. Now w/ the new Mouse Over/ Hover and Full Label tooltip Functionality the user can still see the whole name if needed.

Thanks @metas-rc for the idea/ trigger.

image

@metas-mk metas-mk added this to the Backlog milestone Jul 18, 2017
@Dunkat
Copy link
Contributor

Dunkat commented Jul 19, 2017

I agree with '...' columns will be looks tidy, but it can be annoying for users because always needs hover label to see name. Now user can see part of title and know what there is.

@teosarca
Copy link
Member

@Dunkat yes, but i think those are complementary, i.e.

  1. in case the column caption's is shorten by "..." when hovering user will have the opportunity to see the full name (which sometimes it's important when looking at captions which all start with the same text)
  2. i agree that we shall improve our column captions and use abbreviations or something to make the names shorter but obvious.

@metas-mk
Copy link
Member Author

I had some interesting talks about this topic in the last weeks w/ different users. Most of them said "If the name does not fit into the header but I can hover it and see completely then great! Perhaps I as a user will have to hover a few times before knowing the field content. But after a few times, I will know whats in there and don't have to hover anymore".
--> That's the reason why I think that '...' is more beneficially than 2 additional characters.
Additionally, I agree that we can find some shorter header names, but have to find a balance using the fields in GridView AND detailed view (without using different names!)

@teosarca
Copy link
Member

IMHO this task can be started.

@cadavre cadavre modified the milestones: Backlog, 2017-31 Jul 25, 2017
@Dunkat Dunkat self-assigned this Jul 25, 2017
Dunkat added a commit that referenced this issue Jul 25, 2017
Dunkat added a commit that referenced this issue Jul 25, 2017
cadavre added a commit that referenced this issue Jul 26, 2017
@metas-mk
Copy link
Member Author

metas-mk commented Jul 26, 2017

Tested.

  1. ellipses are shown for header names when too long. OK
  2. Order by triangle is not shown anymore. NOK
  3. Now w/ elipses the gap between end of first columnname to beginning of next column name is too large. Please Reduce. The reductions should also allow more Names to be shown w/o ellipses. If the Gap at the end of field is reserved for the triangle, then leave that at the end of Field and reduce the margin only in front of the Fieldname a bit. NOK
    @Dunkat can u please take care about 2 and 3.

image

@cadavre
Copy link
Contributor

cadavre commented Aug 1, 2017

Ad. 2.) As we discussed on Skype – this is version without arrows ("to see how it looks"). Adding them will cut off more text which makes it completely unreadable. :/ Your decision what to do.
Ad. 3.) It is a cell margin, I believe that reducing it will drastically decrease readability of data. What we can thing about is making margins smaller in table header row. Wdyt? What we will try is make th padding-right .05rem.

Dunkat added a commit that referenced this issue Aug 1, 2017
cadavre added a commit that referenced this issue Aug 1, 2017
@cadavre
Copy link
Contributor

cadavre commented Aug 1, 2017

Update: changed padding-right to smaller.

Dunkat added a commit that referenced this issue Aug 1, 2017
cadavre added a commit that referenced this issue Aug 1, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Aug 4, 2017
[#1111](metasfresh/metasfresh-webui-frontend-legacy#1111)
Lookup Selection Lists not shown in Sequence anymore
[#1098](metasfresh/metasfresh-webui-frontend-legacy#1098)
Cant select last entry in dropdown list
[#2107](#2107) Error:
Missing ESR creditor reference on CAMT 54 Import
[#2106](#2106)
NullPointerException on CAMT 54 Import
[#1089](metasfresh/metasfresh-webui-frontend-legacy#1089)
Avatar Tooltip opens and stays open after selecting Avatar Settings
[#1095](metasfresh/metasfresh-webui-frontend-legacy#1095)
Sales Order Batch Entry with Key Shortcut not working
[#1051](metasfresh/metasfresh-webui-frontend-legacy#1051)
Cut-Off Header names
[#1077](metasfresh/metasfresh-webui-frontend-legacy#1077)
Sitemap Navigation [arrow-up] not working
[#1063](metasfresh/metasfresh-webui-frontend-legacy#1063)
view: when refreshing the page the data is loaded twice
[#1048](metasfresh/metasfresh-webui-frontend-legacy#1048)
POST .../documentView/{windowId} is called twice when opening a view
[#977](metasfresh/metasfresh-webui-frontend-legacy#977)
Add frontend support for modifying the KPI config (caption, offset, etc)
[#2101](#2101) webui -
allow disabling auto-close of picking candidates
[#1771](#1771) add legacy
features

me-45
@metasnw
Copy link
Member

metasnw commented Aug 4, 2017

results from IT:
OK: header in windows
NOK: headers in tabs are not working as requested
Creating follow up: #1117

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

6 participants