|
|
Position ? And content ?
At top of all ?
After menu ? (Strangely : menu/action button are not inside content…)
Inside content ? |
|
|
|
For content
- Surveys : goes to Survey list, OR Home :goes to Welcome page ?
- Survey name (partial + survey ID : Goes to Survey overview
- Survey action page (for Privacy policy settings etc …)
- OR Group name (partial) + group ID + group Step : Group overview
- Group action (except overview)
- OR Question title + question ID + question text (partial) ? : Question overview
|
|
|
|
Please :
- Remind the original issue : https://bugs.limesurvey.org/view.php?id=19178 and https://bugs.limesurvey.org/view.php?id=19516
- Use h1/h2/h3 etc … at the good place and not at random like currently
|
|
|
|
hey hey,
I believe what we're trying to address is the following issue:
"The survey name disappears from ALL pages."
To resolve this, I propose:
- Placing the survey name at the last level of the breadcrumb. This would solve the problem mentioned above.
- Rest can be placed in the three dots ("...") --> display as dropdown/flyout with other nav points
Further I notice:
- Settings --> Participant Settings --> In Breadcrumb it says --> Survey participant settings. There are more inconsistencies like this.
- Settings --> Email templates --> Edit email templates
- I would not display the links under the settings tab in the breadcrumb since the links under the settings tab already solve this.
|
|
|
|
Placing the survey name at the last level of the breadcrumb. This would solve the problem mentioned above.
No : it was the second item of a breadcrumb
And we need to know
- Current group
- Current question
- Current action
on breadcrumb without mouse over etc !
Rest can be placed in the three dots ("...") --> display as dropdown/flyout with other nav points
No !
With 3 dots : the usage of breadcrumb are totally broken ! |
|
|
|
https://designsystem.digital.gov/components/breadcrumb/ |
|
|
|
@DenisChenu
I put the whole specs into the 'Additional information' - I think it largely follows your ideas. Would you be able to code that? |
|
|
|
Seems near this screenshots : https://bugs.limesurvey.org/view.php?id=19552#c80073
Still unsure on best place here , maybe the last one. |
|
|
|
Oups !: about 1st element : Home (/admin/index)
Then
Home > Surveys > Survey name > Privacy policy settings
With links
/index > surveyAdministration/listsurveys > surveyAdministration/view&surveyid=XXX > none |
|
|
|
Behaviour on scrolling up as it is currently in v6.
Before menu then ? |
|
|
|
Yes, like the first screenshot.
I don't think we need "Home" as it can be reached by the logo.
Yes. before menu. |
|
|
|
I don't think we need "Home" as it can be reached by the logo.
:+1: |
|
|
|
@DenisChenu Any progress`? |
|
|
|
No real progress currently … I end the 2 current work (Soft mandatory and XSS issue) and start it.
If one another dev are OK, can take it :) |
|
|
|
@c_schmitz : i see the updated Additional Information : i'm really NOT OK with this !
The default behavior must be
For survey :
- Optionnal (Welcome" maybe with just an icon (to be testes
- Surveys (listing) :
- "[SID] {ellipsed Survey name}" ( goes to Survey overview)
- Action
For Group
- Optionnal (Welcome" maybe with just an icon (to be testes
- Surveys (listing) :
- "[SID] {ellipsed Survey name}" ( goes to Survey overview)
- "[GID] {ellipsed group name}" (goes to group overview)
- Group action
For Question
- Optionnal (Welcome" maybe with just an icon (to be testes
- Surveys (listing) :
- "[SID] {ellipsed Survey name}" ( goes to Survey overview)
- "[GID] {ellipsed group name}" (goes to group overview)
- "[QID] {question title}" (goes to question overview)
- Question action (edit/overview)
WHY have *full ? We always want ellisped Survey name ! Why don't have SID ? It's the IDENTIFIER of the survey ! Except if you have less than 5 it's always needed ! |
|
|
|
This is better for a beginner user, but worse for advanced users who use the breadcrumb to navigate faster.
No ! It's NOT better for a beginner ! |
|
|
|
@DenisChenu, @c_schmitz, in the past months we have migrated many users to LS 6.x. Quite a few of them told us that the missing survey title at the breadcrumb is a really bad UI design. You never know which survey you are currently working on when having multiple tabs open.
Thus, I fully agree "that hiding the current survey name behind the triple dots in the breadcrumb is a bad idea, no matter the user persona.". Can we fix this as a very first thing? |
|
|
|
@mazi : see https://github.com/LimeSurvey/LimeSurvey/pull/3825#issuecomment-2071718713
It's NOT only a breadcrumb issue here. No title on page …
I'm totally OK with your opinion: Usability and ergonomic are really worse on version 6 than on version 5. Some decision on ergonomy are totally broken (replace Add survey button to only a + for another example) |
|
|
|
Well, it definitely won't hurt to also have the title on the page and shown at tabs, I fully agree! |
|
|
|
Updated requirement
Hey Gabriel, so I just spoke with Carsten and this is what we want. There should be an option under admin/user/sa/personalsettings for a user to select short or long version of the breadcrumbs. In the screenshot there is a long version and it should actually look like this:
Survey Name (survey id) / Group title (group id) / Question code - Question text (question id)
The short version should be without IDs in brackets.
The breadcrumbs can cover all the length of the screen, one third of the space max for each part.
Buttons should be set in the second row below with a bit of padding between breadcrumbs and the button row.
For all the text that is too long we just use ellipsis like this "Some long name ...".
And this whole part is already sticky header, so no change necessary here
For the Settings tab, we should use same principle as above, for example
Survey Name (survey id) / General survey settings
but I think we can put survey id for the long and short option, since we have more space
|
|
|
|
PR: https://github.com/LimeSurvey/LimeSurvey/pull/4016 |
|
|
|
Some comments:
Previously, the ellipsis was put by code according to the number of characters. Now it's for CSS. The good thing is that it is more dynamic, and the bad thing is that they are black because they are not part of the <a>. I could fix it but I don't know if it's worth it because you have to modify several things.
Didn't limit each part to 33% because it's not necessarily 3 parts. For example, if you're in the Conditions designer, it's 4 parts.
Took out the first part (the "Surveys" part) because understood that was wanted, but not totally sure. |
|
|
|
@gabrieljenik I think the question should be with ellipsis as well (see screenshot). |
|
|
|
- must remove GoogleSheetsConnector
- bad support of text-wrap-mode (explain @tibor.pacalat issue surely)
- avoid empty link (add nbsp or another string)
- a title can help (and avoid empty link for A11Y)
- A11Y issue with links
|
|
