Skip to content

Commit

Permalink
Notifications set tab-index to -1
Browse files Browse the repository at this point in the history
  • Loading branch information
annagav committed Oct 23, 2024
1 parent 061b03d commit 276d735
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions authentication/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -271,6 +271,7 @@ class RegisterEmailSerializer(SocialAuthSerializer):
next = serializers.CharField(write_only=True, required=False)

def validate(self, attrs):
print("here")
token = (attrs.get("partial", {}) or {}).get("token", None)
email = attrs.get("email", None)
if not email and not token:
Expand Down
2 changes: 1 addition & 1 deletion frontend/public/scss/notification.scss
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.notifications {
position: relative;
z-index: 1;
z-index: -1;

// HACK: Using this rule to get around styling issues with <p> elements in the
// notification content (notification content in Wagtail is configured to be HTML,
Expand Down
2 changes: 1 addition & 1 deletion frontend/public/src/components/NotificationContainer.js
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ export class NotificationContainer extends React.Component<Props, State> {

return (
<div
className="notifications order-2"
className="notifications order-2 tab-index-1"
id="notifications-container"
ref={this.headingRef}
>
Expand Down

0 comments on commit 276d735

Please sign in to comment.