Skip to content

Commit 9bd5f97

Browse files
committed
UI adjuestment
1 parent 0502a21 commit 9bd5f97

File tree

3 files changed

+6
-18
lines changed

3 files changed

+6
-18
lines changed

options/locale/locale_en-US.ini

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1600,7 +1600,7 @@ pulls.merge_instruction_hint = `You can also view <a class="show-instruction">co
16001600
pulls.merge_instruction_step1_desc = From your project repository, check out a new branch and test the changes.
16011601
pulls.merge_instruction_step2_desc = Merge the changes and update on Gitea.
16021602

1603-
pulls.auto_merge_button_title = When Checks Succeed
1603+
pulls.auto_merge_button_when_succeed = (When checks succeed)
16041604
pulls.auto_merge_when_succeed = Auto merge when all checks succeed
16051605
pulls.auto_merge_newly_scheduled = The pull request was scheduled to merge when all checks succeed.
16061606
pulls.auto_merge_has_pending_schedule = %[1]s scheduled this pull request to auto merge when all checks succeed %[2]s.

templates/repo/issue/view_content/pull.tmpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@
342342
'baseLink': {{.Link}},
343343
'textCancel': {{$.i18n.Tr "cancel"}},
344344
'textDeleteBranch': {{$.i18n.Tr "repo.branch.delete" .HeadTarget}},
345-
'textAutoMergeButtonTitle': {{$.i18n.Tr "repo.pulls.auto_merge_button_title"}},
345+
'textAutoMergeButtonWhenSucceed': {{$.i18n.Tr "repo.pulls.auto_merge_button_when_succeed"}},
346346
'textAutoMergeWhenSucceed': {{$.i18n.Tr "repo.pulls.auto_merge_when_succeed"}},
347347
'textAutoMergeCancelSchedule': {{$.i18n.Tr "repo.pulls.auto_merge_cancel_schedule"}},
348348

web_src/js/components/PullRequestMergeForm.vue

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
<button class="ui button" :class="mergeButtonStyleClass" type="submit" name="do" :value="mergeStyle">
3232
{{ mergeStyleDetail.textDoMerge }}
3333
<template v-if="autoMergeWhenSucceed">
34-
({{ mergeForm.textAutoMergeButtonTitle }})
34+
({{ mergeForm.textAutoMergeButtonWhenSucceed }})
3535
</template>
3636
</button>
3737

@@ -54,7 +54,7 @@
5454
<span class="button-text">
5555
{{ mergeStyleDetail.textDoMerge }}
5656
<template v-if="autoMergeWhenSucceed">
57-
({{ mergeForm.textAutoMergeButtonTitle }})
57+
{{ mergeForm.textAutoMergeButtonWhenSucceed }}
5858
</template>
5959
</span>
6060
</button>
@@ -78,14 +78,9 @@
7878
<!-- if can NOT merge now, only show one action "auto merge when succeed" -->
7979
<div class="item" v-if="msd.allowed && !mergeForm.canMergeNow && !msd.hideAutoMerge" :key="msd.name" @click.stop="switchMergeStyle(msd.name, true)">
8080
<div class="action-text">
81-
{{ msd.textDoMerge }}
82-
</div>
83-
<div class="auto-merge-full">
84-
<svg-icon name="octicon-clock" :size="14"/>
85-
<span class="auto-merge-text">{{ mergeForm.textAutoMergeWhenSucceed }}</span>
81+
{{ msd.textDoMerge }} {{ mergeForm.textAutoMergeButtonWhenSucceed }}
8682
</div>
8783
</div>
88-
8984
</template>
9085
</div>
9186
</div>
@@ -211,14 +206,7 @@ export default {
211206
padding: 0.8rem;
212207
flex: 1
213208
}
214-
.auto-merge-full {
215-
display: flex;
216-
align-items: center;
217-
justify-content: center;
218-
}
219-
.auto-merge-full .auto-merge-text {
220-
padding: 0 1rem 0 0.25rem;
221-
}
209+
222210
.auto-merge-small {
223211
width: 40px;
224212
display: flex;

0 commit comments

Comments
 (0)