diff options
Diffstat (limited to 'app/views')
-rw-r--r-- | app/views/devise/registrations/new.html.haml | 2 | ||||
-rw-r--r-- | app/views/devise/sessions/new.html.haml | 2 | ||||
-rw-r--r-- | app/views/devise/shared/_experimental_separate_sign_up_flow_box.html.haml | 5 | ||||
-rw-r--r-- | app/views/devise/shared/_signin_box.html.haml | 2 | ||||
-rw-r--r-- | app/views/layouts/_head.html.haml | 2 | ||||
-rw-r--r-- | app/views/layouts/devise_experimental_separate_sign_up_flow.html.haml | 3 | ||||
-rw-r--r-- | app/views/profiles/show.html.haml | 1 | ||||
-rw-r--r-- | app/views/registrations/welcome.html.haml | 17 |
8 files changed, 25 insertions, 9 deletions
diff --git a/app/views/devise/registrations/new.html.haml b/app/views/devise/registrations/new.html.haml index dfdf7429dc5..5f85235e8fa 100644 --- a/app/views/devise/registrations/new.html.haml +++ b/app/views/devise/registrations/new.html.haml @@ -1,5 +1,5 @@ - page_title "Sign up" -- if use_experimental_separate_sign_up_flow? +- if experiment_enabled?(:signup_flow) = render 'devise/shared/experimental_separate_sign_up_flow_box' - else = render 'devise/shared/signup_box' diff --git a/app/views/devise/sessions/new.html.haml b/app/views/devise/sessions/new.html.haml index 17e5da501f5..8f6c3ecbe58 100644 --- a/app/views/devise/sessions/new.html.haml +++ b/app/views/devise/sessions/new.html.haml @@ -4,7 +4,7 @@ - if form_based_providers.any? = render 'devise/shared/tabs_ldap' - else - - unless use_experimental_separate_sign_up_flow? + - unless experiment_enabled?(:signup_flow) = render 'devise/shared/tabs_normal' .tab-content - if password_authentication_enabled_for_web? || ldap_enabled? || crowd_enabled? diff --git a/app/views/devise/shared/_experimental_separate_sign_up_flow_box.html.haml b/app/views/devise/shared/_experimental_separate_sign_up_flow_box.html.haml index f92c29da5e6..5d163d03c73 100644 --- a/app/views/devise/shared/_experimental_separate_sign_up_flow_box.html.haml +++ b/app/views/devise/shared/_experimental_separate_sign_up_flow_box.html.haml @@ -1,4 +1,4 @@ -- max_name_length = 128 +- content_for(:page_title, _('Register for GitLab')) - max_username_length = 255 .signup-box.p-3.mb-2 .signup-body @@ -6,9 +6,6 @@ .devise-errors.mt-0 = render "devise/shared/error_messages", resource: resource = invisible_captcha - .name.form-group - = f.label :name, _('Full name'), class: 'label-bold' - = f.text_field :name, class: "form-control top js-block-emoji js-validate-length", :data => { :max_length => max_name_length, :max_length_message => s_("SignUp|Name is too long (maximum is %{max_length} characters).") % { max_length: max_name_length }, :qa_selector => 'new_user_name_field' }, required: true, title: _("This field is required.") .username.form-group = f.label :username, class: 'label-bold' = f.text_field :username, class: "form-control middle js-block-emoji js-validate-length js-validate-username", :data => { :max_length => max_username_length, :max_length_message => s_("SignUp|Username is too long (maximum is %{max_length} characters).") % { max_length: max_username_length }, :qa_selector => 'new_user_username_field' }, pattern: Gitlab::PathRegex::NAMESPACE_FORMAT_REGEX_JS, required: true, title: _("Please create a username with only alphanumeric characters.") diff --git a/app/views/devise/shared/_signin_box.html.haml b/app/views/devise/shared/_signin_box.html.haml index deaceccbfc7..746d43edbad 100644 --- a/app/views/devise/shared/_signin_box.html.haml +++ b/app/views/devise/shared/_signin_box.html.haml @@ -23,7 +23,7 @@ .login-body = render 'devise/sessions/new_base' -- if use_experimental_separate_sign_up_flow? +- if experiment_enabled?(:signup_flow) %p.light.mt-2 = _("Don't have an account yet?") = link_to _("Register now"), new_registration_path(:user) diff --git a/app/views/layouts/_head.html.haml b/app/views/layouts/_head.html.haml index 1efd8647a67..b8c9f0ae1e8 100644 --- a/app/views/layouts/_head.html.haml +++ b/app/views/layouts/_head.html.haml @@ -5,7 +5,7 @@ -# anything from this page beforehand. -# Part of an experiment to build a new sign up flow. Will be removed again with -# https://gitlab.com/gitlab-org/growth/engineering/issues/64 -- if use_experimental_separate_sign_up_flow? && current_path?("sessions#new") +- if experiment_enabled?(:signup_flow) && current_path?("sessions#new") = javascript_tag nonce: true do :plain if (window.location.hash === '#register-pane') { diff --git a/app/views/layouts/devise_experimental_separate_sign_up_flow.html.haml b/app/views/layouts/devise_experimental_separate_sign_up_flow.html.haml index b10145b62af..2f05717fc0e 100644 --- a/app/views/layouts/devise_experimental_separate_sign_up_flow.html.haml +++ b/app/views/layouts/devise_experimental_separate_sign_up_flow.html.haml @@ -14,7 +14,8 @@ = render_if_exists 'layouts/devise_help_text' .text-center.signup-heading.mt-3.mb-3 = image_tag(image_url('logo.svg'), class: 'gitlab-logo', alt: 'GitLab Logo') - %h2= _('Register for GitLab.com') + - if content_for?(:page_title) + %h2= yield :page_title = yield %hr.footer-fixed .footer-container diff --git a/app/views/profiles/show.html.haml b/app/views/profiles/show.html.haml index 0576f51fa83..68b7efc6fb4 100644 --- a/app/views/profiles/show.html.haml +++ b/app/views/profiles/show.html.haml @@ -94,6 +94,7 @@ - else = f.text_field :name, label: s_('Profiles|Full name'), required: true, title: s_("Profiles|Using emojis in names seems fun, but please try to set a status message instead"), wrapper: { class: 'col-md-9 qa-full-name rspec-full-name' }, help: s_("Profiles|Enter your name, so people you know can recognize you") = f.text_field :id, readonly: true, label: s_('Profiles|User ID'), wrapper: { class: 'col-md-3' } + = f.select :role, ::User.roles.keys.map { |role| [role.titleize, role] }, {}, class: 'input-md' = render_if_exists 'profiles/email_settings', form: f = f.text_field :skype, class: 'input-md', placeholder: s_("Profiles|username") diff --git a/app/views/registrations/welcome.html.haml b/app/views/registrations/welcome.html.haml new file mode 100644 index 00000000000..02ab974ecc0 --- /dev/null +++ b/app/views/registrations/welcome.html.haml @@ -0,0 +1,17 @@ +- content_for(:page_title, _('Welcome to GitLab<br>%{username}!' % { username: html_escape(current_user.username) }).html_safe) +- max_name_length = 128 +.text-center.mb-3 + = _('In order to tailor your experience with GitLab<br>we would like to know a bit more about you.').html_safe +.signup-box.p-3.mb-2 + .signup-body + = form_for(current_user, url: users_sign_up_update_role_path, html: { class: 'new_new_user gl-show-field-errors', 'aria-live' => 'assertive' }) do |f| + .devise-errors.mt-0 + = render 'devise/shared/error_messages', resource: current_user + .name.form-group + = f.label :name, _('Full name'), class: 'label-bold' + = f.text_field :name, class: 'form-control top js-block-emoji js-validate-length', :data => { :max_length => max_name_length, :max_length_message => s_('Name is too long (maximum is %{max_length} characters).') % { max_length: max_name_length }, :qa_selector => 'new_user_name_field' }, required: true, title: _('This field is required.') + .form-group + = f.label :role, _('Role'), class: 'label-bold' + = f.select :role, ::User.roles.keys.map { |role| [role.titleize, role] }, {}, class: 'form-control' + .submit-container.mt-3 + = f.submit _('Get started!'), class: 'btn-register btn btn-block mb-0 p-2' |