Skip to content

Commit

Permalink
fix: remove inner scroll from BaseError (#20104)
Browse files Browse the repository at this point in the history
  • Loading branch information
ZachJW34 authored Feb 11, 2022
1 parent 5c5f5d5 commit faa8fb5
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
8 changes: 5 additions & 3 deletions packages/frontend-shared/src/components/Alert.vue
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,11 @@
lazy
:initially-open="initiallyOpen"
:disable="!canCollapse"
class="rounded-t rounded-b outline-none overflow-hidden group"
class="overflow-hidden rounded-t rounded-b outline-none group"
:class="[
classes.headerClass,
{[`hocus-default border-1 border-transparent rounded ${classes.ring}`]: canCollapse}]"
height="300"
:max-height="maxHeight"
>
<template #target="{ open }">
<div
Expand Down Expand Up @@ -98,7 +98,8 @@ const props = withDefaults(defineProps<{
dismissible?: boolean,
collapsible?: boolean,
modelValue?: boolean,
iconClasses?: string
iconClasses?: string,
maxHeight?: string,
}>(), {
title: undefined,
modelValue: true,
Expand All @@ -107,6 +108,7 @@ const props = withDefaults(defineProps<{
headerClass: undefined,
bodyClass: undefined,
iconClasses: '',
maxHeight: undefined,
})
const title = computed(() => props.title ?? 'Alert')
Expand Down
5 changes: 3 additions & 2 deletions packages/launchpad/src/error/BaseError.vue
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<template>
<div class="mx-auto space-y-32px text-center min-w-476px max-w-848px pt-16px children:text-center">
<div class="mx-auto space-y-32px text-center min-w-476px max-w-848px py-16px children:text-center">
<div>
<h1
class="font-medium leading-snug text-32px text-gray-900"
Expand Down Expand Up @@ -30,6 +30,7 @@
header-class="bg-red-100 text-red-600 rounded-b-none"
:icon="ErrorOutlineIcon"
icon-classes="icon-dark-red-400"
max-height="none"
>
<p
v-if="errorMessage"
Expand All @@ -43,7 +44,7 @@
>
<Collapsible
disable
max-height="400px"
max-height="none"
initially-open
>
<template #target="{open, toggle}">
Expand Down

3 comments on commit faa8fb5

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on faa8fb5 Feb 11, 2022

Choose a reason for hiding this comment

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

Circle has built the linux x64 version of the Test Runner.

Learn more about this pre-release platform-specific build at https://on.cypress.io/installing-cypress#Install-pre-release-version.

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/10.0.0/linux-x64/circle-10.0-release-faa8fb5302ae8fac4ac53f1dcd214abe35bce3ab/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on faa8fb5 Feb 11, 2022

Choose a reason for hiding this comment

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

Circle has built the win32 x64 version of the Test Runner.

Learn more about this pre-release platform-specific build at https://on.cypress.io/installing-cypress#Install-pre-release-version.

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/10.0.0/win32-x64/circle-10.0-release-faa8fb5302ae8fac4ac53f1dcd214abe35bce3ab/cypress.tgz

@cypress-bot
Copy link
Contributor

@cypress-bot cypress-bot bot commented on faa8fb5 Feb 11, 2022

Choose a reason for hiding this comment

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

Circle has built the darwin x64 version of the Test Runner.

Learn more about this pre-release platform-specific build at https://on.cypress.io/installing-cypress#Install-pre-release-version.

Run this command to install the pre-release locally:

npm install https://cdn.cypress.io/beta/npm/10.0.0/darwin-x64/circle-10.0-release-faa8fb5302ae8fac4ac53f1dcd214abe35bce3ab/cypress.tgz

Please sign in to comment.