Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It should be possible to adjust the sizes of the panels by dragging them narrower or wider #12787

Closed
Tracked by #11103 ...
Annikenkbrathen opened this issue May 10, 2024 · 12 comments · Fixed by #13044
Closed
Tracked by #11103 ...
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-specification Status: Used for issues that are ready for functional decription og detailed design. user-need Issues describing user needs ux UX help needed

Comments

@Annikenkbrathen
Copy link

Annikenkbrathen commented May 10, 2024

The user should be able to control the space usage here and adjust the panels. The preview and component panels should also be able to be hidden if the user desires

Skjermbilde 2024-05-10 kl  08 50 15
@Annikenkbrathen Annikenkbrathen added ux UX help needed area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-specification Status: Used for issues that are ready for functional decription og detailed design. labels May 10, 2024
@nkylstad nkylstad added the user-need Issues describing user needs label May 29, 2024
@nkylstad nkylstad added the good first issue Issues that are easy to fix for new team members or external contributors label Jun 7, 2024
@wrt95
Copy link
Contributor

wrt95 commented Jun 12, 2024

Create new hook useAppLocalStorage that uses app and org and uses the useLocalStorage hook 😄
We must also create a @stduio/component named ResizeableLayout that can be reused on dataModelPage, textEditorPage and processEditorPage(?)

@wrt95
Copy link
Contributor

wrt95 commented Jun 12, 2024

Linked with #12730, and should be implemented in the same way

@Jondyr Jondyr self-assigned this Jun 18, 2024
@Jondyr Jondyr moved this to 👷 In Progress in Team Studio Jun 18, 2024
@nkylstad nkylstad removed the good first issue Issues that are easy to fix for new team members or external contributors label Jul 2, 2024
@Jondyr Jondyr moved this from 👷 In Progress to 🔎 Review in Team Studio Jul 4, 2024
@Jondyr Jondyr assigned Jondyr and unassigned Jondyr Jul 4, 2024
@Jondyr Jondyr moved this from 🔎 Review to 👷 In Progress in Team Studio Jul 4, 2024
@Jondyr Jondyr moved this from 👷 In Progress to 🔎 Review in Team Studio Jul 4, 2024
@Jondyr Jondyr removed their assignment Jul 4, 2024
@mlqn mlqn assigned mlqn and Jondyr and unassigned mlqn Jul 9, 2024
@Jondyr Jondyr linked a pull request Jul 11, 2024 that will close this issue
5 tasks
@Jondyr Jondyr moved this from 🔎 Review to 🧪 Test in Team Studio Jul 11, 2024
@Jondyr Jondyr removed their assignment Jul 11, 2024
@ErlingHauan ErlingHauan self-assigned this Jul 15, 2024
@ErlingHauan
Copy link
Contributor

Testing

Nice work on a feature that is super nice to finally have in Studio 👏

Should we add a min-width for the two middle columns in the Lage page, so that it is not possible to hide them entirely?
Not having a min-width can lead to the issue below, where it gets difficult to retrieve the two middle columns, and the "Open preview" button sticks to the left side of the screen instead of the right side.

20240715-0646-17.2517201.mp4

This one is more a personal opinion, but I think this feature would look cleaner without the hover effect on the resize handle. To me, the Studio UI appears more solid without it. What do you think? Here's a video where the hover effect is disabled:

20240715-0703-59.5373789.mp4

Other than this, it looks like it works perfectly, both in Datamodell and Lage 😊

@ErlingHauan ErlingHauan assigned Jondyr and unassigned ErlingHauan Jul 15, 2024
@ErlingHauan ErlingHauan moved this from 🧪 Test to 👀 Test feedback in Team Studio Jul 15, 2024
@standeren
Copy link
Contributor

I also have two small comments on this one;

  1. maybe also very personal preference, but could the default be to show the preview?
  2. If I use a small screen, regular 14", and I open preview, the content of the config tab gets very large/wide so I get a horizontal scroll inside it 🫣
Skjermopptak.2024-07-15.kl.10.58.07.mov

But need to add that this was a nice feature to finally get in Studio 🤩 It will be even better as soon as we store the chosen sizes 👏

@lassopicasso
Copy link
Contributor

This looks really nice! Love this feature! I have a small comment on what @ErlingHauan mentioned:

Should we add a min-width for the two middle columns in the Lage page, so that it is not possible to hide them entirely?

In the refinement of this task, didn't we discuss having the same effect as in the preview? When the config and pages columns get narrow enough, shouldn't they be hidden with a button, similar to the preview?

@Jondyr
Copy link
Member

Jondyr commented Jul 15, 2024

Testing

Nice work on a feature that is super nice to finally have in Studio 👏

Should we add a min-width for the two middle columns in the Lage page, so that it is not possible to hide them entirely? Not having a min-width can lead to the issue below, where it gets difficult to retrieve the two middle columns, and the "Open preview" button sticks to the left side of the screen instead of the right side.

20240715-0646-17.2517201.mp4
This one is more a personal opinion, but I think this feature would look cleaner without the hover effect on the resize handle. To me, the Studio UI appears more solid without it. What do you think? Here's a video where the hover effect is disabled:

20240715-0703-59.5373789.mp4
Other than this, it looks like it works perfectly, both in Datamodell and Lage 😊

@ErlingHauan Good catch on the preview collapsing issue, i agree, a min-width on the center columns make sense 👍 For the on-hover effect, i am fairly neutral to the two options, but I do see your point on it feeling more solid. I will make changes for both.

@Jondyr
Copy link
Member

Jondyr commented Jul 15, 2024

@standeren

  1. maybe also very personal preference, but could the default be to show the preview?

I agree, especially since collapsed state is not persistent.

  1. If I use a small screen, regular 14", and I open preview, the content of the config tab gets very large/wide so I get a horizontal scroll inside it 🫣

Thanks for noticing this. It is definitely an unintended issue that the content accordion's content is not resizing as it used to before.

@Jondyr
Copy link
Member

Jondyr commented Jul 15, 2024

@standeren @ErlingHauan
I've made some changes and created a PR here: #13154
Here's a short clip showing some of the changes:

test-feedback-resizable.mp4

@Jondyr Jondyr removed their assignment Jul 15, 2024
@Jondyr Jondyr moved this from 👀 Test feedback to 🧪 Test in Team Studio Jul 15, 2024
@ErlingHauan ErlingHauan moved this from 🧪 Test to ✅ Done in Team Studio Jul 17, 2024
@ErlingHauan ErlingHauan removed the status in Team Studio Jul 17, 2024
@ErlingHauan ErlingHauan moved this to 🧪 Test in Team Studio Jul 17, 2024
@ErlingHauan
Copy link
Contributor

I noticed one more thing: it is not possible to click and drag the scrollbars any more:

20240717-0705-47.6720153.mp4

@ErlingHauan ErlingHauan moved this from 🧪 Test to 👀 Test feedback in Team Studio Jul 17, 2024
@Jondyr
Copy link
Member

Jondyr commented Jul 17, 2024

I noticed one more thing: it is not possible to click and drag the scrollbars any more:

20240717-0705-47.6720153.mp4

This is interesting, it works in firefox as the scrollbar is different and wider. I have tried to find a solution that does not alter the scrollbar or make the draggable handle area very small, but have found no solutions yet.

We could possibly solve this by styling the scrollbar on webkit-based browsers wider, as we are using ::-webkit-scrollbar in App.css. But the mdn docs explicitly state "Non-standard: This feature is non-standard and is not on a standards track. Do not use it on production sites facing the Web: it will not work for every user. There may also be large incompatibilities between implementations and the behavior may change in the future.". However, removing this styling is quite intrusive:

screenshots

image
image

Firefox

image

Maybe this should be discussed?

@ErlingHauan
Copy link
Contributor

Ouch, that doesn't look very good! 😄

According to this article, Firefox, Chrome, and Edge are starting to support the official scrollbar standard, while Safari is lagging behind.

Maybe we should revisit how we style the scrollbars in a separate issue?

@Jondyr
Copy link
Member

Jondyr commented Jul 17, 2024

Ouch, that doesn't look very good! 😄

According to this article, Firefox, Chrome, and Edge are starting to support the official scrollbar standard, while Safari is lagging behind.

Maybe we should revisit how we style the scrollbars in a separate issue?

Good find 👍 Agreed, i think we should revisit this. I created a new issue referencing your comment.

@Jondyr Jondyr closed this as completed Jul 17, 2024
@github-project-automation github-project-automation bot moved this from 👀 Test feedback to 🧪 Test in Team Studio Jul 17, 2024
@Jondyr Jondyr moved this from 🧪 Test to ✅ Done in Team Studio Jul 17, 2024
@Jondyr Jondyr removed their assignment Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui-editor Area: Related to the designer tool for assembling app UI in Altinn Studio. status/ready-for-specification Status: Used for issues that are ready for functional decription og detailed design. user-need Issues describing user needs ux UX help needed
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

8 participants