Skip to content

Align Tickets and Talk menu design with Common Dashboard Menu #1547

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.

Sign up for GitHub

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

Open
prayag78 wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Align Tickets and Talk menu design with Common Dashboard Menu #1547

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bbe332e
Align Tickets and Talk menus with Common Dashboard
prayag78 Dec 18, 2025
9848fce
layout adjustment
prayag78 Dec 19, 2025
97a6801
update ui
prayag78 Dec 20, 2025
759e5c9
fix spacing
prayag78 Dec 20, 2025
17fde8c
remove whitespace
prayag78 Dec 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 8 additions & 4 deletions app/eventyay/orga/templates/orga/event/dashboard.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,14 @@
</div>
{% endif %}
<div id="main-title" class="d-md-flex justify-content-between">
<h2>
{{ request.event.name }}
<small class="text-muted small">{{ request.event.get_date_range_display }}</small>
</h2>
<div class="navigation-title">
<h2>
{{ request.event.name }}
<small class="event-date text-muted">
{{ request.event.get_date_range_display }}
</small>
</h2>
</div>
{% include "orga/event/component_link.html" %}
</div>
<div id="timeline-steps" class="stages">
Expand Down
15 changes: 15 additions & 0 deletions app/eventyay/static/eventyay-common/scss/custom.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
font-size: 16px;
font-weight: normal;
border-radius: 0;
padding: 7px 7px;
}
.header-nav.active {
background-color: $hover-button-color !important;
Expand All @@ -16,6 +17,8 @@
display: flex;
flex-wrap: wrap;
align-items: center;
text-align: center;
font-weight: 200;
.navigation-button {
margin-left: auto;
}
Expand Down Expand Up @@ -157,6 +160,18 @@
}
}

@media (min-width: $screen-lg-min) {
#event-nav .navigation .navigation-button {
margin-top: 10px;
}
}

@media (max-width: $screen-sm-max) {
#event-nav .navigation {
margin-top: 18px;
}
}

@media (min-width: $screen-sm-min) and (max-width: $screen-sm-max) {
.dashboard .widget-container.widget-small.no-padding.column {
width: 100%;
Expand Down
32 changes: 31 additions & 1 deletion app/eventyay/static/orga/css/_layout.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ body {
background: var(--color-bg);
flex-grow: 1;
max-width: 100%;
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

The change from padding: 20px 40px to padding: 18px 44px introduces asymmetric adjustments (-2px vertical, +4px horizontal). This creates a non-uniform scaling pattern that may be difficult to maintain. Consider using a consistent scaling factor or documenting the specific design rationale for these particular values to aid future maintenance.

Suggested change
max-width: 100%;
max-width: 100%;
/* Note: Asymmetric padding (18px vertical, 44px horizontal) is intentional to match surrounding layout spacing. */

Copilot uses AI. Check for mistakes.
padding: 20px 40px;
padding: 18px 44px;
margin: 0;
margin-top: 50px;
height: calc(100vh - 50px);
Expand Down Expand Up @@ -567,6 +567,9 @@ nav.navbar,
.header-nav.btn-success,
.header-nav .btn-success {
border-radius: 0 !important;
font-size: 16px;
font-weight: normal;
padding: 6px 10px;
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

Inconsistent padding values between similar button styles. In _layout.css, the padding is set to "6px 10px", while in custom.scss the padding is "7px 7px". These should be aligned for consistency across the dashboard menus.

Suggested change
padding: 6px 10px;
padding: 7px 7px;

Copilot uses AI. Check for mistakes.
}

/* Navigation button hover, focus, and active states */
Expand All @@ -586,6 +589,33 @@ nav.navbar,
box-shadow: inset 0 3px 5px rgba(0, 0, 0, 0.125);
}

Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

Consider adding a comment explaining the purpose of this style block and its relationship to the common dashboard navigation styles. This would help future maintainers understand why these specific styles are needed for the event dashboard.

Suggested change
/* Event dashboard main title area
* These styles control the layout of the main header/title section
* for the event dashboard. The flex container aligns the page title
* with the navigation button, and works together with the common
* header navigation button styles defined above to create a cohesive
* dashboard header.
*/

Copilot uses AI. Check for mistakes.
#main-title {
display: flex;
flex-wrap: wrap;
font-weight: 200;
margin-bottom: 20px;
}

#main-title .navigation-button {
margin-left: auto;
}

#main-title .navigation-title {
text-align: center;
}

#main-title .navigation-title h2 {
font-size: 30px;
font-weight: 500;
line-height: 1.2;
}

#main-title .navigation-title h2 .event-date {
font-size: 65%;
font-weight: 400;
line-height: 1;
}

.speaker-info {
display: flex;
flex-direction: row;
Expand Down