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.

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

register: added info box for institutional email #1101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

register: added info box for institutional email #1101

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
9 changes: 9 additions & 0 deletions assets/less/zenodo-rdm/modules/popup.overrides
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,3 +48,12 @@
max-width: 80vw;
white-space: break-spaces;
}

.field-info{
padding: 0.4em !important;
margin-top: -0.7em !important;
[data-tooltip]::after {
width: 25vw;
white-space: break-spaces;
}
}
Comment on lines +51 to +59
Copy link
Member

@ptamarit ptamarit Feb 21, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

It's probably fine, but wondering if this is the right file to put this CSS rule.

15 changes: 14 additions & 1 deletion templates/semantic-ui/invenio_accounts/register_user.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,16 @@
{%- extends config.ACCOUNTS_COVER_TEMPLATE %}

{% from "invenio_accounts/_macros.html" import render_field, form_errors %}

{% macro render_field_info(message, tooltip) %}
<div class="ui small bottom attached message info field-info">
{{ message }}
<span class="ui icon" data-position="bottom center" data-tooltip="{{ tooltip }}">
<i class="circle info icon"></i>
</span>
</div>
{% endmacro %}

{% block page_body %}
<div class="ui centered container">
<h1 class="ui inverted header text-center">Research. Shared! Sign up today</h1>
Expand Down Expand Up @@ -52,7 +62,10 @@ <h1 class="ui inverted header text-center">Research. Shared! Sign up today</h1>
{{ form.hidden_tag() }}

{%- block registration_form_fields scoped %}
{{ render_field(form.email, icon="user icon", autofocus=True, errormsg=False) }}
{{ render_field(form.email, icon="user icon", autofocus=True, errormsg=False) }}
{{ render_field_info(_('Please use an institutional email address'), _('An institutional email address helps us verify your account and prevents our spam filtering system from automatically blocking your account.')) }}
Copy link
Member

Choose a reason for hiding this comment

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

@lnielsen is that the final text we want to have on the tooltip? I don't remember if we decided on something else.

Copy link
Member

Choose a reason for hiding this comment

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

@0einstein0 do you know if we can go ahead with this specific text?



{{ render_field(form.password, icon="lock icon", errormsg=False) }}

{%- if form.password_confirm %}
Expand Down