fix(presale): Fix Tickets page layout inconsistencies

[ArnavBallinCode]

• Align logo flush left and remove auto-rounded corners
• Align user info to the right edge
• Improve language selector styling (inline with separators for <= 3 languages)
• Fix dropdown text visibility and alignment

Screen.Recording.2025-11-30.at.4.13.17.PM.mov

Fixes #1364

[Copilot]

Pull request overview

This PR fixes layout inconsistencies on the Tickets page by improving the positioning and styling of the logo, user info, and language selector components.

• Logo alignment changed to flush left with auto-rounded corners removed
• User info and language selector aligned to the right edge using flexbox
• Language selector now has conditional rendering: inline display for ≤3 languages, dropdown for >3 languages

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 7 comments.

File	Description
app/eventyay/static/pretixpresale/scss/custom.scss	Added styles for inline language selector, dropdown language selector with visibility improvements, header content flexbox alignment, and responsive adjustments for mobile devices
app/eventyay/static/pretixpresale/scss/_event.scss	Changed logo alignment to left, removed border-radius styling, and added proper display properties
app/eventyay/presale/templates/pretixpresale/fragment_login_status.html	Simplified navigation structure by removing dynamic margin-right calculation and improving indentation
app/eventyay/presale/templates/pretixpresale/event/base.html	Restructured logo placement logic to work with different theme backgrounds, implemented conditional language selector rendering (dropdown vs inline), and duplicated login status inclusion in two sections

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The loop to find the current language name is inefficient. Consider using a filter or template variable to get the current language name directly instead of iterating through all languages every time the dropdown button is rendered. This creates O(n) operations for every page load when it should be O(1).

Suggested change

	<button class="btn btn-default dropdown-toggle language-selector" type="button" id="languageDropdown2" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
	{% for l in languages %}
	{% if l.code == request.LANGUAGE_CODE %}
	{{ l.name_local }}
	{% endif %}
	{% endfor %}
	<span class="caret"></span>
	</button>
	{% with current_language=None %}
	{% for l in languages %}
	{% if l.code == request.LANGUAGE_CODE %}
	{% with current_language=l %}
	{% break %}
	{% endwith %}
	{% endif %}
	{% endfor %}
	<button class="btn btn-default dropdown-toggle language-selector" type="button" id="languageDropdown2" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
	{{ current_language.name_local }}
	<span class="caret"></span>
	</button>
	{% endwith %}

[Copilot]

The z-index value of 10000 is extremely high and could cause stacking context issues with other elements. Consider using a more reasonable value (e.g., 1000 or 100) unless there's a specific requirement for such a high value. The child dropdown menu's z-index of 10001 would then also be reduced proportionally.

[Copilot]

The duplicate color value #fff on line 103 overrides the color set on line 102. The second declaration makes the first one redundant. Remove line 102 or line 103 to eliminate this duplication.

Suggested change

color: #fff;

[Copilot]

The dropdown button lacks accessible text for screen readers when the dropdown is expanded. While aria-expanded is set to "false" in the initial state, it should be dynamically updated to "true" when the dropdown opens. Consider adding JavaScript to toggle this attribute or ensure the Bootstrap dropdown plugin handles this automatically.

[Copilot]

The dropdown button lacks accessible text for screen readers when the dropdown is expanded. While aria-expanded is set to "false" in the initial state, it should be dynamically updated to "true" when the dropdown opens. Consider adding JavaScript to toggle this attribute or ensure the Bootstrap dropdown plugin handles this automatically.

[Copilot]

The loop to find the current language name is inefficient. Consider using a filter or template variable to get the current language name directly instead of iterating through all languages every time the dropdown button is rendered. This creates O(n) operations for every page load when it should be O(1).

Suggested change

	<button class="btn btn-default dropdown-toggle language-selector" type="button" id="languageDropdown" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
	{% for l in languages %}
	{% if l.code == request.LANGUAGE_CODE %}
	{{ l.name_local }}
	{% endif %}
	{% endfor %}
	<span class="caret"></span>
	</button>
	{% with current_language=None %}
	{% for l in languages %}
	{% if l.code == request.LANGUAGE_CODE %}
	{% with current_language=l %}
	{% break %}
	{% endwith %}
	{% endif %}
	{% endfor %}
	<button class="btn btn-default dropdown-toggle language-selector" type="button" id="languageDropdown" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
	{{ current_language.name_local }}
	<span class="caret"></span>
	</button>
	{% endwith %}

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.