Skip to content

Commit 6330dc4

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent de122df commit 6330dc4

File tree

34 files changed

+235
-49
lines changed

34 files changed

+235
-49
lines changed

.gitlab/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1252,6 +1252,7 @@ lib/gitlab/checks/**
12521252
[Localization Team] @gitlab-com/localization/maintainers
12531253
/doc-locale/**
12541254
/doc/development/i18n/proofreader.md
1255+
/argo_translation.yml
12551256

12561257
[Authorization] @gitlab-org/software-supply-chain-security/authorization/approvers
12571258
/config/initializers/declarative_policy.rb

GITLAB_KAS_VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
b8a5f8c7440eb511a10903d7f3d1862c96cd6428
1+
82d009755f8dff37a144d63220fdf70c1643c166

app/assets/javascripts/admin/broadcast_messages/components/message_form.vue

Lines changed: 36 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import {
99
GlFormInput,
1010
GlFormSelect,
1111
GlFormTextarea,
12+
GlModal,
1213
} from '@gitlab/ui';
1314
import axios from '~/lib/utils/axios_utils';
1415
import { s__, __ } from '~/locale';
@@ -43,6 +44,7 @@ export default {
4344
GlFormInput,
4445
GlFormSelect,
4546
GlFormTextarea,
47+
GlModal,
4648
},
4749
directives: {
4850
SafeHtml,
@@ -85,6 +87,12 @@ export default {
8587
showInCliDescription: s__(
8688
'BroadcastMessages|Show the broadcast message in a command-line interface as a Git remote response',
8789
),
90+
confirm: {
91+
title: s__('BroadcastMessages|Security Notice'),
92+
text: s__(
93+
'BroadcastMessages|Broadcast messages must not contain sensitive information. All broadcast messages are publicly accessible through the API, regardless of path or role scoping.',
94+
),
95+
},
8896
},
8997
messageThemes: THEMES,
9098
messageTypes: TYPES,
@@ -238,14 +246,31 @@ export default {
238246
}
239247
return TARGET_ALL;
240248
},
249+
confirmSubmit() {
250+
this.$refs.confirmModal.show();
251+
},
241252
},
242253
safeHtmlConfig: {
243254
ADD_TAGS: ['use'],
244255
},
256+
modal: {
257+
actionPrimary: {
258+
text: s__('BroadcastMessages|I understand and confirm'),
259+
attributes: {
260+
variant: 'confirm',
261+
},
262+
},
263+
actionSecondary: {
264+
text: __('Cancel'),
265+
attributes: {
266+
variant: 'default',
267+
},
268+
},
269+
},
245270
};
246271
</script>
247272
<template>
248-
<gl-form @submit.prevent="onSubmit">
273+
<gl-form @submit.prevent="confirmSubmit">
249274
<gl-broadcast-message
250275
class="gl-my-6"
251276
:type="type"
@@ -366,5 +391,15 @@ export default {
366391
{{ $options.i18n.cancel }}
367392
</gl-button>
368393
</div>
394+
395+
<gl-modal
396+
ref="confirmModal"
397+
modal-id="confirm-modal"
398+
:title="$options.i18n.confirm.title"
399+
:action-primary="$options.modal.actionPrimary"
400+
:action-secondary="$options.modal.actionSecondary"
401+
@primary="onSubmit"
402+
>{{ $options.i18n.confirm.text }}</gl-modal
403+
>
369404
</gl-form>
370405
</template>

app/assets/javascripts/packages_and_registries/package_registry/components/details/package_files.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ export default {
513513
/>
514514
</template>
515515
516-
<template #footer>
516+
<template v-if="$scopedSlots.upload" #footer>
517517
<slot name="upload" :refetch="refetchPackageFiles"></slot>
518518
</template>
519519

app/assets/javascripts/projects/pipelines/charts/components/app.vue

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<script>
22
import { GlTabs, GlTab } from '@gitlab/ui';
3+
import MigrationAlert from 'ee_component/analytics/dora/components/migration_alert.vue';
34
import { mergeUrlParams, updateHistory, getParameterValues } from '~/lib/utils/url_utility';
45
import { __, s__ } from '~/locale';
56
import { InternalEvents } from '~/tracking';
@@ -13,9 +14,14 @@ export default {
1314
components: {
1415
GlTabs,
1516
GlTab,
17+
MigrationAlert,
1618
},
1719
mixins: [InternalEvents.mixin(), glFeatureFlagsMixin()],
1820
inject: {
21+
projectPath: {
22+
type: String,
23+
default: '',
24+
},
1925
shouldRenderDoraCharts: {
2026
type: Boolean,
2127
default: false,
@@ -43,7 +49,7 @@ export default {
4349
},
4450
];
4551
46-
if (this.shouldRenderDoraCharts) {
52+
if (this.shouldRenderDoraCharts && !this.glFeatures.doraMetricsDashboard) {
4753
tabs.push(
4854
{
4955
key: 'deployment-frequency',
@@ -93,6 +99,11 @@ export default {
9399
tabs,
94100
};
95101
},
102+
computed: {
103+
showDoraMetricsMigrationAlert() {
104+
return this.shouldRenderDoraCharts && this.glFeatures.doraMetricsDashboard;
105+
},
106+
},
96107
created() {
97108
this.syncActiveTab();
98109
window.addEventListener('popstate', this.syncActiveTab);
@@ -118,6 +129,12 @@ export default {
118129
</script>
119130
<template>
120131
<div>
132+
<migration-alert
133+
v-if="showDoraMetricsMigrationAlert"
134+
:namespace-path="projectPath"
135+
is-project
136+
/>
137+
121138
<gl-tabs v-if="tabs.length > 1" :value="activeTabIndex" @input="onTabInput">
122139
<gl-tab
123140
v-for="tab in tabs"

app/assets/javascripts/rapid_diffs/toggle_file/adapter.js

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,13 @@ function expand(root = this.diffElement) {
2323
root.querySelector('[data-file-body]').hidden = false;
2424
}
2525

26+
function stopTransition(element) {
27+
element.style.transition = 'none';
28+
requestAnimationFrame(() => {
29+
element.style.transition = '';
30+
});
31+
}
32+
2633
export const ToggleFileAdapter = {
2734
clicks: {
2835
toggleFile(event, button) {
@@ -32,7 +39,10 @@ export const ToggleFileAdapter = {
3239
} else {
3340
collapse.call(this);
3441
}
35-
getOppositeToggleButton(button).focus();
42+
const oppositeButton = getOppositeToggleButton(button);
43+
oppositeButton.focus();
44+
// a replaced button triggers another transition that we need to stop
45+
stopTransition(oppositeButton);
3646
},
3747
},
3848
[EXPAND_FILE]() {

app/assets/stylesheets/components/rapid_diffs/diff_file_component.scss

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,10 @@
1-
@import 'framework/variables';
2-
31
.rd-diff-file-component {
42
scroll-margin-top: var(--rd-app-sticky-top-with-padding);
53
}
64

75
.rd-diff-file {
8-
padding-bottom: $gl-padding;
9-
106
--rd-diff-file-border-radius: #{calc($gl-border-radius-base - 1px)};
7+
padding-bottom: $gl-padding;
118
}
129

1310
.rd-diff-file-header {
@@ -18,8 +15,7 @@
1815
display: flex;
1916
background-color: var(--gl-background-color-subtle);
2017
border: 1px solid var(--gl-border-color-default);
21-
padding: $gl-padding-8 $gl-padding;
22-
min-height: px-to-rem($file-header-height);
18+
padding: $gl-spacing-scale-3 $gl-spacing-scale-4;
2319
border-radius: var(--rd-diff-file-border-radius) var(--rd-diff-file-border-radius) 0 0;
2420
word-break: break-word;
2521
z-index: 1;
@@ -46,7 +42,7 @@
4642
}
4743

4844
.rd-diff-file-toggle {
49-
margin-right: $gl-spacing-scale-2;
45+
margin-right: $gl-spacing-scale-3;
5046
}
5147

5248
.rd-diff-file[data-collapsed] .rd-diff-file-toggle-button[data-opened],

app/assets/stylesheets/components/rapid_diffs/text_file_viewers.scss

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,10 @@
111111
}
112112

113113
// Do not use nesting here to allow quick jump to code based on the selector
114-
.rd-line-number:hover:not(:empty) {
114+
.rd-line-number:hover:not(:empty),
115+
.rd-hunk-lines-inline:hover .rd-line-number,
116+
.rd-hunk-lines:has(.rd-line-content[data-position=new]:hover:not(:empty)) .rd-line-number[data-position=new],
117+
.rd-hunk-lines:has(.rd-line-content[data-position=old]:hover:not(:empty)) .rd-line-number[data-position=old] {
115118
background-color: var(--code-line-nubmer-hover-background-color, $gl-color-purple-100);
116119
border-color: var(--code-line-nubmer-hover-border-color, $gl-color-purple-200);
117120
}

app/assets/stylesheets/framework/header.scss

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@
8080
margin-left: -4px;
8181
height: 32px;
8282
width: auto;
83-
border-radius: $gl-border-radius-base;
83+
@apply gl-rounded-base;
8484

8585
&:hover,
8686
&:focus {
@@ -106,7 +106,7 @@
106106
appearance: none;
107107
border: 0;
108108
background-color: transparent;
109-
border-radius: $gl-border-radius-base;
109+
@apply gl-rounded-base;
110110
}
111111

112112
.header-logged-out-dropdown {
@@ -125,7 +125,7 @@
125125
display: inline-block;
126126
padding: 6px 8px;
127127
height: 32px;
128-
border-radius: $gl-border-radius-base;
128+
@apply gl-rounded-base;
129129
color: var(--gl-color-neutral-100);
130130
font-weight: $gl-font-weight-normal;
131131
font-size: $gl-font-size;

app/assets/stylesheets/framework/popup.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
padding: $gl-padding;
1010
background-color: $gray-10;
1111
border: 1px solid $gray-50;
12-
border-radius: $gl-border-radius-base;
12+
@apply gl-rounded-base;
1313
box-shadow: 0 5px 8px $popup-box-shadow-color;
1414
position: relative;
1515
}

0 commit comments

Comments
 (0)