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(core/executions): Disable text selection when re-ordering pipelines #6603

Merged

Conversation

erikmunson
Copy link
Member

When re-ordering pipelines in the filter sidebar, it can get kinda wonky if you accidentally make the browser think you're selecting text. If you manage to click + drag just right, sometimes it even thinks you're trying to drag a multi-line text selection for copy-pasting.

This switches off text selection when entering re-ordering mode for strategies and normal pipelines. Outside re-ordering mode you can still select the pipeline names to copy-paste, if that's something you're into.

Copy link
Contributor

@anotherchrisberry anotherchrisberry left a comment

Choose a reason for hiding this comment

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

Didn't know about that CSS property, neat

@erikmunson
Copy link
Member Author

yeah it's great for making user-hostile interfaces without the fuss:

* {
  user-select: none;
}

@erikmunson erikmunson merged commit 70fb404 into spinnaker:master Feb 22, 2019
@erikmunson erikmunson deleted the disable-selection-pipeline-reorder branch February 22, 2019 01:21
erikmunson added a commit to erikmunson/deck that referenced this pull request Feb 25, 2019
3e75815 refactor(core): migrate momentjs functionality to luxon + date-fns (spinnaker#6604)
754c540 Merge branch 'master' into lb-is-loading
70fb404 fix(core/executions): Disable text selection when re-ordering pipelines (spinnaker#6603)
a35b4cb Merge branch 'master' into lb-is-loading
1d320d6 Merge branch 'master' into lb-is-loading
c026514 Merge branch 'master' into lb-is-loading
cf528dc fix(core): loading spinner for LBs not dismissed
erikmunson added a commit that referenced this pull request Feb 25, 2019
#6615)

* chore(core): Bump version to 0.0.339

3e75815 refactor(core): migrate momentjs functionality to luxon + date-fns (#6604)
754c540 Merge branch 'master' into lb-is-loading
70fb404 fix(core/executions): Disable text selection when re-ordering pipelines (#6603)
a35b4cb Merge branch 'master' into lb-is-loading
1d320d6 Merge branch 'master' into lb-is-loading
c026514 Merge branch 'master' into lb-is-loading
cf528dc fix(core): loading spinner for LBs not dismissed

* chore(amazon): Bump version to 0.0.177

510557f fix(amazon/serverGroup): Do not apply default AZs unless the user wants to usePreferredZones (#6609)

* chore(titus): Bump version to 0.0.77

5e77299 fix(titus): Help users with iam profile setting (#6552)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants