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: make scrollbars draggable when offsets are set #2916

Merged
merged 3 commits into from
Feb 4, 2021

Conversation

christophepas
Copy link
Contributor

@christophepas christophepas commented Feb 3, 2021

Cf #2915 (comment)

Today when setting offsetLeft or offsetRight when using the package library, scrollbars will display, but can't be dragged. This is an attempt to fix it.

@vercel
Copy link

vercel bot commented Feb 3, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/excalidraw/excalidraw/bxh2n4b1m
✅ Preview: https://excalidraw-git-fork-christophepas-fix-scrollbar-w-offset.excalidraw.vercel.app

@lipis
Copy link
Member

lipis commented Feb 3, 2021

In unrelated issue.. I think we should include pull requests here as well:

Maybe that's why actions from external branches are not running (sometimes).

@dwelle
Copy link
Member

dwelle commented Feb 4, 2021

Thanks for the fix!

@dwelle dwelle merged commit dbae33e into excalidraw:master Feb 4, 2021
lipis added a commit that referenced this pull request Feb 4, 2021
* 'master' of github.com:excalidraw/excalidraw:
  fix: make scrollbars draggable when offsets are set (#2916)
  chore: Run actions on pull requests as well (#2917)
  feat: Add idle detection to collaboration feature (#2877)
  fix: pointer-events being disabled on free-draw (#2912)
  feat: don't store to LS during collab (#2909)
lipis added a commit that referenced this pull request Feb 4, 2021
* 'master' of github.com:excalidraw/excalidraw: (58 commits)
  chore: Update translations from Crowdin (#2906)
  fix: toolbar unnecessarily eats too much width (#2924)
  fix: mistakenly hardcoding scale (#2925)
  fix: text editor not visible in dark mode (#2920)
  feat: support supplying custom scale when exporting canvas (#2904)
  fix: incorrect z-index of text editor (#2914)
  feat: Show version in the stats dialog (#2908)
  fix: make scrollbars draggable when offsets are set (#2916)
  chore: Run actions on pull requests as well (#2917)
  feat: Add idle detection to collaboration feature (#2877)
  fix: pointer-events being disabled on free-draw (#2912)
  feat: don't store to LS during collab (#2909)
  chore: Update translations from Crowdin (#2898)
  fix: track zenmode and grid mode usage (#2900)
  refactor: Use the latest vercel configuration instead of now (#2893)
  chore: Update translations from Crowdin (#2894)
  Update i18n.ts
  Update locales-coverage-description.js
  feat: add view mode in Excalidraw (#2840)
  chore(deps): bump @sentry/integrations from 6.0.1 to 6.0.3 (#2889)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants