You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by bb...@apache.org on 2022/04/06 17:26:42 UTC
[airflow] 02/03: Fix button sizes
This is an automated email from the ASF dual-hosted git repository.
bbovenzi pushed a commit to branch inrcease-grid-font
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit a835cca82005fcc5ab6b8096b86670a7be88eb93
Author: Brent Bovenzi <br...@gmail.com>
AuthorDate: Wed Apr 6 12:45:01 2022 -0400
Fix button sizes
---
.../js/tree/details/content/taskInstance/ExtraLinks.jsx | 1 +
.../www/static/js/tree/details/content/taskInstance/Logs.jsx | 2 ++
.../tree/details/content/taskInstance/taskActions/Clear.jsx | 12 ++++++------
.../details/content/taskInstance/taskActions/MarkFailed.jsx | 10 +++++-----
.../details/content/taskInstance/taskActions/MarkSuccess.jsx | 10 +++++-----
.../js/tree/details/content/taskInstance/taskActions/Run.jsx | 5 ++++-
6 files changed, 23 insertions(+), 17 deletions(-)
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/ExtraLinks.jsx b/airflow/www/static/js/tree/details/content/taskInstance/ExtraLinks.jsx
index b9b60b4c01..c4da5778ab 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/ExtraLinks.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/ExtraLinks.jsx
@@ -50,6 +50,7 @@ const ExtraLinks = ({
as={Link}
colorScheme="blue"
href={url}
+ fontSize="lg"
isDisabled={!url}
target={isExternal(url) ? '_blank' : undefined}
>
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/Logs.jsx b/airflow/www/static/js/tree/details/content/taskInstance/Logs.jsx
index c89e1d3364..6495f95563 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/Logs.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/Logs.jsx
@@ -61,6 +61,7 @@ const Logs = ({
key={index}
href={fullExternalUrl}
target="_blank"
+ fontSize="lg"
>
{index}
</LinkButton>,
@@ -78,6 +79,7 @@ const Logs = ({
// eslint-disable-next-line react/no-array-index-key
key={index}
href={fullMetadataUrl}
+ fontSize="lg"
>
{index === 0 ? 'All' : index}
</LinkButton>
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Clear.jsx b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Clear.jsx
index 5cd45ef6fb..497de9f065 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Clear.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Clear.jsx
@@ -102,12 +102,12 @@ const Run = ({
return (
<Flex justifyContent="space-between" width="100%">
<ButtonGroup isAttached variant="outline">
- <ActionButton bg={past && 'gray.100'} onClick={onTogglePast} name="Past" />
- <ActionButton bg={future && 'gray.100'} onClick={onToggleFuture} name="Future" />
- <ActionButton bg={upstream && 'gray.100'} onClick={onToggleUpstream} name="Upstream" />
- <ActionButton bg={downstream && 'gray.100'} onClick={onToggleDownstream} name="Downstream" />
- <ActionButton bg={recursive && 'gray.100'} onClick={onToggleRecursive} name="Recursive" />
- <ActionButton bg={failed && 'gray.100'} onClick={onToggleFailed} name="Failed" />
+ <ActionButton bg={past && 'gray.100'} fontSize="lg" onClick={onTogglePast} name="Past" />
+ <ActionButton bg={future && 'gray.100'} fontSize="lg" onClick={onToggleFuture} name="Future" />
+ <ActionButton bg={upstream && 'gray.100'} fontSize="lg" onClick={onToggleUpstream} name="Upstream" />
+ <ActionButton bg={downstream && 'gray.100'} fontSize="lg" onClick={onToggleDownstream} name="Downstream" />
+ <ActionButton bg={recursive && 'gray.100'} fontSize="lg" onClick={onToggleRecursive} name="Recursive" />
+ <ActionButton bg={failed && 'gray.100'} fontSize="lg" onClick={onToggleFailed} name="Failed" />
</ButtonGroup>
<Button
colorScheme="blue"
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkFailed.jsx b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkFailed.jsx
index 098707476c..819bfda5e9 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkFailed.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkFailed.jsx
@@ -93,11 +93,11 @@ const MarkFailed = ({
return (
<Flex justifyContent="space-between" width="100%">
- <ButtonGroup isAttached variant="outline" fontSize="lg">
- <ActionButton bg={past && 'gray.100'} onClick={onTogglePast} name="Past" />
- <ActionButton bg={future && 'gray.100'} onClick={onToggleFuture} name="Future" />
- <ActionButton bg={upstream && 'gray.100'} onClick={onToggleUpstream} name="Upstream" />
- <ActionButton bg={downstream && 'gray.100'} onClick={onToggleDownstream} name="Downstream" />
+ <ButtonGroup isAttached variant="outline">
+ <ActionButton bg={past && 'gray.100'} fontSize="lg" onClick={onTogglePast} name="Past" />
+ <ActionButton bg={future && 'gray.100'} fontSize="lg" onClick={onToggleFuture} name="Future" />
+ <ActionButton bg={upstream && 'gray.100'} fontSize="lg" onClick={onToggleUpstream} name="Upstream" />
+ <ActionButton bg={downstream && 'gray.100'} fontSize="lg" onClick={onToggleDownstream} name="Downstream" />
</ButtonGroup>
<Button fontSize="lg" colorScheme="red" onClick={onClick} isLoading={isMarkLoading || isConfirmLoading}>
Mark Failed
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkSuccess.jsx b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkSuccess.jsx
index fc8783ab50..5cc7ec68f1 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkSuccess.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/MarkSuccess.jsx
@@ -87,11 +87,11 @@ const Run = ({ dagId, runId, taskId }) => {
return (
<Flex justifyContent="space-between" width="100%">
- <ButtonGroup isAttached variant="outline" fontSize="lg">
- <ActionButton bg={past && 'gray.100'} onClick={onTogglePast} name="Past" />
- <ActionButton bg={future && 'gray.100'} onClick={onToggleFuture} name="Future" />
- <ActionButton bg={upstream && 'gray.100'} onClick={onToggleUpstream} name="Upstream" />
- <ActionButton bg={downstream && 'gray.100'} onClick={onToggleDownstream} name="Downstream" />
+ <ButtonGroup isAttached variant="outline">
+ <ActionButton bg={past && 'gray.100'} fontSize="lg" onClick={onTogglePast} name="Past" />
+ <ActionButton bg={future && 'gray.100'} fontSize="lg" onClick={onToggleFuture} name="Future" />
+ <ActionButton bg={upstream && 'gray.100'} fontSize="lg" onClick={onToggleUpstream} name="Upstream" />
+ <ActionButton bg={downstream && 'gray.100'} fontSize="lg" onClick={onToggleDownstream} name="Downstream" />
</ButtonGroup>
<Button fontSize="lg" colorScheme="green" onClick={onClick} isLoading={isMarkLoading || isConfirmLoading}>
Mark Success
diff --git a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Run.jsx b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Run.jsx
index 725412fe04..4f07fe137d 100644
--- a/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Run.jsx
+++ b/airflow/www/static/js/tree/details/content/taskInstance/taskActions/Run.jsx
@@ -52,11 +52,12 @@ const Run = ({
return (
<Flex justifyContent="space-between" width="100%">
- <ButtonGroup isAttached variant="outline" fontSize="lg">
+ <ButtonGroup isAttached variant="outline">
<Button
bg={ignoreAllDeps && 'gray.100'}
onClick={onToggleAllDeps}
title="Ignores all non-critical dependencies, including task state and task_deps"
+ fontSize="lg"
>
Ignore All Deps
</Button>
@@ -64,6 +65,7 @@ const Run = ({
bg={ignoreTaskState && 'gray.100'}
onClick={onToggleTaskState}
title="Ignore previous success/failure"
+ fontSize="lg"
>
Ignore Task State
</Button>
@@ -71,6 +73,7 @@ const Run = ({
bg={ignoreTaskDeps && 'gray.100'}
onClick={onToggleTaskDeps}
title="Disregard the task-specific dependencies, e.g. status of upstream task instances and depends_on_past"
+ fontSize="lg"
>
Ignore Task Deps
</Button>