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

Fix click-and-hold scrolling in Designer #13194

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Conversation

ErlingHauan
Copy link
Contributor

@ErlingHauan ErlingHauan commented Jul 19, 2024

Description

  • Disabled the left side of the resize handle so that it doesn't interfere with the scrollbar.
  • Made the same change to the horizontal axis, so the behaviour is the same in case we want to use the other axis in the future.
  • Added 1px width to the scrollbar in webkit browsers in Designer. It's hardly noticable for anyone, but makes it a little bit easier to click and drag.

This solution was inspired by Rider/Windows, which appears to work the same way:

fixScrollbars.mp4

Related Issue(s)

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)

@github-actions github-actions bot added the solution/studio/designer Issues related to the Altinn Studio Designer solution. label Jul 19, 2024
@ErlingHauan ErlingHauan marked this pull request as ready for review July 19, 2024 11:32
@ErlingHauan ErlingHauan requested a review from Jondyr July 19, 2024 11:32
@Jondyr Jondyr closed this Jul 22, 2024
@Jondyr Jondyr reopened this Jul 22, 2024
Copy link
Member

@Jondyr Jondyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix!

@Jondyr Jondyr removed their assignment Jul 22, 2024
@Jondyr Jondyr merged commit 12719e1 into main Jul 22, 2024
12 checks passed
@Jondyr Jondyr deleted the fix/scroll-and-resize-handle branch July 22, 2024 07:43
Jondyr added a commit that referenced this pull request Jul 29, 2024
)

* Change `AppPreviewBar` LegacySelect to StudioNativeSelect

* Change `SecurityLevelSelect` LegacySelect to StudioNativeSelect

* Change `LangSelector` LegacySelect to StudioNativeSelect

* Remove selected property from <option>

* Change `SchemaInspector` LegacySelect to StudioNativeSelect

* Change `PolicyCardRules` LegacySelect to StudioNativeSelect

* Use defaultValue instead of selected on <option>

* Change `TextResourceEditor` LegacySelect to StudioNativeSelect

* Change `componentSpecificContent` LegacySelect to StudioNativeSelect

* Delete unused code

* Change `componentconfig` LegacySelect to StudioNativeSelect

Remove unused import

* Fix some tests broken after LegacySelect->StudioNativeSelect swap

Fix some tests broken after LegacySelect->StudioNativeSelect swap

* Fix failing `each()` tests

* Replace LegacyTextField with StudioTextfield in schemainspector

* Fix css after replacing legacy components

* Replace LegacyTextField and fix possible bug with saving float numbers

Bug: saving float numbers for min/max restrictions on number components

* Fix formatting

* Replace LegacyTextField for ux-editor, delete unused code

* Fix tests after replacing LegacyTextField

* Fix interesting test failure, new component allows exponents like 1e2

"1test 2" becomes "1e2" = 1*10^2=100

* Fix tests using wrong role for number input

* Widen test for coverage

* Add tests for combobox enum with multiple selection

Remove unused code

Fix accidental deletion of test value

* Fix formatting, prettier

* Merge main test fixes

* Fix playwright test failing after legacy component replacement

* fixed overfloee in datamodellknytninger accordion

* fixes overflow in datamodel bilnding in featureFlag

* Replace LegacyPopover for ReleaseContainer

* Replace LegacyPopover for datamodel deletion button

* Replace LegacyPopover for AltinnConfirmationDialog

* Change AltinnConfirmationDialogProps to use StudioPopoverProps

* Replace LegacyPopover for ThreeDotsMenu

* Replace LegacyPopover in PreviewLimitationsInfo

* Replace LegacyTable for TextList

* Fix click-and-hold scrolling in designer (#13194)

* Fix dashboard URL in Playwright (#13203)

* eid logger deploy env setup (#13205)

* Update dependency org.springframework.boot:spring-boot-starter-parent to v3.3.2 (#13198)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* change helm repository (#13208)

* pass bools as strings  in helm chart(#13209)

* Add missing ids to nativeselect, required for label `for`

* Fix playwright tests failing after LegacySelect replacement

Also change value assertion to follow playwright best practices:

**NOTE** If you need to assert input value, prefer
[expect(locator).toHaveValue(value[,
options])](https://playwright.dev/docs/api/class-locatorassertions#locator-assertions-to-have-value)
to avoid flakiness. See [assertions
guide](https://playwright.dev/docs/test-assertions) for more details.

Fix typo

* Make sure ids for tablerowtype is unique in a multi type list

* Replace LegacyToggleButtonGroup on the preview page

* Fix app preview tests for togglegroup

button -> radio

* Replace legacyfieldset for mapcomponent

* Replace LegacyTextArea in ItemDataComponent

* Update import from storybook entry for resizablelayout

* Replace legacy components for ux-editorv3

* Remove design-system-react dependency

Squash: Also commit `yarn.lock`

* Fix linting issues

* Update SelectDataModelComponent.test.tsx

* Fix informative message variant validation issue

* Fix some event handling collisions between new and old components

* Remove commented unused css

* Merge main

* Use `useId` hook to generate unique fallback id for StudioNativeSelect

* Revert change to button disabled property check in langselector test

* Add name option back to getRole query

---------

Co-authored-by: JamalAlabdullah <[email protected]>
Co-authored-by: Erling Hauan <[email protected]>
Co-authored-by: Tomas Engebretsen <[email protected]>
Co-authored-by: Mirko Sekulic <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solution/studio/designer Issues related to the Altinn Studio Designer solution.
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants