Skip to content
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.

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

[7942] euth, euth_wagtail: fix styling for login #2677

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

m4ra
Copy link
Contributor

@m4ra m4ra commented Jan 18, 2024

fixes #2676

@m4ra m4ra force-pushed the mk-2024-01-18-fix-dropdown-login branch from 2e68856 to ac56afa Compare January 18, 2024 16:56
@m4ra m4ra requested a review from hom3mad3 January 18, 2024 16:57
@m4ra
Copy link
Contributor Author

m4ra commented Jan 18, 2024

@hom3mad3 once this is merged I can initiate the release process.

@m4ra
Copy link
Contributor Author

m4ra commented Jan 22, 2024

@hom3mad3 can you have a look when possible, I would like to work on the OPIN release after that.

</li>
-->
</ul>
</ul>
Copy link

@hom3mad3 hom3mad3 Jan 22, 2024

Choose a reason for hiding this comment

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

this can go<ul>

Copy link
Contributor Author

@m4ra m4ra Jan 22, 2024

Choose a reason for hiding this comment

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

  • the last </ul> is closing the list tag that starts at the beginning --> <ul class="nav navbar-nav navbar-right ...>
  • the line 4 class dropdown is needed for when we have logged in, see the if statement that indicates if user.is_authenticated

@hom3mad3

Copy link
Contributor Author

Choose a reason for hiding this comment

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

uhm.. you are right, we don't need the extra closing

@m4ra m4ra force-pushed the mk-2024-01-18-fix-dropdown-login branch from ac56afa to b27d1e4 Compare January 22, 2024 15:20
Copy link

@hom3mad3 hom3mad3 left a comment

Choose a reason for hiding this comment

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

looks ok! ideally, we should fix the entire file, but it works for hiding the signup!

@hom3mad3 hom3mad3 merged commit 53adc5c into main Jan 22, 2024
3 checks passed
@hom3mad3 hom3mad3 deleted the mk-2024-01-18-fix-dropdown-login branch January 22, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#7942 [OPIN] Stop the registration of new accounts
2 participants