gem "actionpack-page_caching", ">= 1.2.0"
gem "activerecord-import"
gem "active_record_union"
-gem "bootstrap", "~> 4.5.0"
-gem "bootstrap_form", "~> 4.0"
+gem "bootstrap", "~> 5.1.0"
+gem "bootstrap_form", "~> 5.0"
gem "cancancan"
gem "composite_primary_keys", "~> 14.0.0"
gem "config"
debug_inspector (>= 0.0.1)
bootsnap (1.12.0)
msgpack (~> 1.2)
- bootstrap (4.5.3)
+ bootstrap (5.1.3)
autoprefixer-rails (>= 9.1.0)
- popper_js (>= 1.14.3, < 2)
+ popper_js (>= 2.9.3, < 3)
sassc-rails (>= 2.0.0)
- bootstrap_form (4.5.0)
+ bootstrap_form (5.1.0)
actionpack (>= 5.2)
activemodel (>= 5.2)
brakeman (5.2.3)
parser (3.1.2.0)
ast (~> 2.4.1)
pg (1.4.1)
- popper_js (1.16.0)
+ popper_js (2.9.3)
progress (3.6.0)
public_suffix (4.0.7)
puma (5.6.4)
better_errors
binding_of_caller
bootsnap (>= 1.4.2)
- bootstrap (~> 4.5.0)
- bootstrap_form (~> 4.0)
+ bootstrap (~> 5.1.0)
+ bootstrap_form (~> 5.0)
brakeman
browser
bzip2-ffi
.appendTo($linkSection);
$("<div>")
- .attr("class", "form-check form-group")
+ .attr("class", "form-check mb-3")
.appendTo($form)
.append(
$("<label>")
.appendTo($imageSection);
$("<div>")
- .attr("class", "form-group form-check")
+ .attr("class", "mb-3 form-check")
.appendTo($form)
.append(
$("<label>")
.icon.note.grey { /* no-r2 */ background-position: -240px -20px; }
.icon.query { /* no-r2 */ background-position: -260px 0; }
-/* Rules for links */
-
-a {
- color: #24d;
- text-decoration: none;
- outline: 0;
- &:hover {
- text-decoration: underline;
- }
-}
-
/* Utility for de-emphasizing content */
.text-muted a {
.login-menu {
.btn-outline-secondary {
- @include button-outline-variant($darkgrey);
+ @include button-outline-variant($darkgrey, $white, null, null, $white);
}
}
}
}
+/* Stop bootstrap 5 from floating legends when they don't need to be */
+legend {
+ float: none;
+}
+
+/* Override the text colour for primary and secondary buttons, to match our
+ bootstrap 4 colours. Note this has accessibility issues, which is why
+ bootstrap 5 calculates black as the appropriate colour, and we should
+ reconsider our colours at some point with that in mind. */
+
+.btn-primary {
+ @include button-variant($primary, $primary, $white, null, null, $white, null, null, $white, null, null, $white);
+}
+
+.btn-secondary {
+ @include button-variant($secondary, $secondary, $white, null, null, $white, null, null, $white, null, null, $white);
+}
+
+.btn-outline-secondary {
+ @include button-outline-variant($secondary, $white, null, null, $white);
+}
+
/* Rules for the search and direction forms */
header .search_forms,
}
}
-/* Customise the background colour of striped tables */
-
-.table-striped > tbody > tr:nth-child(2n+1) > td,
-.table-striped > tbody > tr:nth-child(2n+1) > th {
- background-color: $offwhite;
-}
-
/* Rules for OpenID logo */
.openid_logo {
$keyline: 1px solid $lightgrey;
$list-highlight: #FFFFC0;
$border: 1px solid $grey;
+
+$link-color: #24d;
+$link-hover-color: #24d;
+$link-decoration: none;
+$link-hover-decoration: underline;
+
+$table-striped-bg: $offwhite;
+
+$enable-negative-margins: true;
<%= f.password_field :pass_crypt, :value => "", :autocomplete => "new-password" %>
<%= f.password_field :pass_crypt_confirmation, :value => "", :autocomplete => "new-password" %>
- <fieldset class="form-group">
- <label for="user_auth_provider"><%= t(".external auth") %></label>
- <div class="form-row">
+ <fieldset class="mb-3">
+ <label for="user_auth_provider" class="form-label"><%= t(".external auth") %></label>
+ <div class="row">
<%= f.select(:auth_provider, Auth.providers, :hide_label => true, :wrapper => { :class => "col-auto mb-0" }) %>
<%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }) %>
</div>
<small class="form-text text-muted">(<a href="<%= t ".openid.link" %>" target="_new"><%= t ".openid.link text" %></a>)</small>
</fieldset>
- <div class="form-group">
- <label><%= t ".public editing.heading" %></label>
+ <div class="mb-3">
+ <label class="form-label"><%= t ".public editing.heading" %></label>
<span class="form-text text-muted">
<% if current_user.data_public? %>
<%= t ".public editing.enabled" %>
</span>
</div>
- <div class="form-group">
- <label><%= t ".contributor terms.heading" %></label>
+ <div class="mb-3">
+ <label class="form-label"><%= t ".contributor terms.heading" %></label>
<span class="form-text text-muted">
<% if current_user.terms_agreed? %>
<%= t ".contributor terms.agreed" %>
<% if current_user %>
<% unless @changeset.open? %>
<form action="#" class="mb-3">
- <div class="form-group">
+ <div class="mb-3">
<textarea class="form-control" name="text" cols="40" rows="5"></textarea>
</div>
<div>
<form action="#">
<input type="hidden" name="lon" autocomplete="off">
<input type="hidden" name="lat" autocomplete="off">
- <div class="form-group">
+ <div class="mb-3">
<textarea class="form-control" name="text" cols="40" rows="10" maxlength="2000" placeholder="<%= t("javascripts.notes.new.advice") %>"></textarea>
</div>
<div class="buttons clearfix">
<% if @note.status == "open" %>
<% if current_user -%>
<form action="#">
- <div class="form-group">
+ <div class="mb-3">
<textarea class="form-control" name="text" cols="40" rows="5" maxlength="2000"></textarea>
</div>
<div class="btn-wrapper">
-<div class="user_popup row no-gutters mx-1">
+<div class="user_popup row g-0 mx-1">
<div class="col-auto mx-1">
<%= user_thumbnail popup %>
</div>
<%= tag.div "", :id => "map", :class => "border border-grey rounded", :data => { :lat => @lat, :lon => @lon, :zoom => @zoom } %>
- <div class="form-row">
+ <div class="row">
<%= f.text_field :latitude, :wrapper_class => "col-sm-4", :id => "latitude" %>
<%= f.text_field :longitude, :wrapper_class => "col-sm-4", :id => "longitude" %>
<div class="col-sm-4">
<p><%= t ".search_guidance" %></p>
<%= form_tag(issues_path, :method => :get) do %>
- <div class="form-row">
- <div class="form-group col-md-auto">
+ <div class="row gx-1">
+ <div class="mb-3 col-md-auto">
<%= select_tag :status,
options_for_select(Issue.aasm.states.map(&:name).map { |state| [t(".states.#{state}"), state] }, params[:status]),
:include_blank => t(".select_status"),
:data => { :behavior => "category_dropdown" },
- :class => "form-control custom-select" %>
+ :class => "form-select" %>
</div>
- <div class="form-group col-md-auto">
+ <div class="mb-3 col-md-auto">
<%= select_tag :issue_type,
options_for_select(@issue_types, params[:issue_type]),
:include_blank => t(".select_type"),
:data => { :behavior => "category_dropdown" },
- :class => "form-control custom-select" %>
+ :class => "form-select" %>
</div>
- <div class="form-group col-md">
+ <div class="mb-3 col-md">
<%= text_field_tag :search_by_user,
params[:search_by_user],
:placeholder => t(".reported_user"),
:autocomplete => "on",
:class => "form-control" %>
</div>
- <div class="form-group col-md-auto">
+ <div class="mb-3 col-md-auto">
<%= select_tag :last_updated_by,
options_for_select(@users.all.collect { |f| [f.display_name, f.id] } << [t(".not_updated"), "nil"], params[:last_updated_by]),
:include_blank => t(".select_last_updated_by"),
:data => { :behavior => "category_dropdown" },
- :class => "form-control custom-select" %>
+ :class => "form-select" %>
</div>
- <div class="form-group col-md-auto">
+ <div class="mb-3 col-md-auto">
<%= submit_tag t(".search"), :name => nil, :class => "btn btn-primary" %>
</div>
</div>
:class => "btn btn-outline-primary geolink editlink",
:id => "editanchor",
:data => { :editor => preferred_editor } %>
- <a class='btn btn-outline-primary dropdown-toggle dropdown-toggle-split' data-toggle='dropdown' href='#'></a>
+ <a class='btn btn-outline-primary dropdown-toggle dropdown-toggle-split' data-bs-toggle='dropdown' href='#'></a>
<ul class='dropdown-menu'>
<% Editors::RECOMMENDED_EDITORS.each do |editor| %>
<li>
<%= link_to t("layouts.about"), about_path, :class => "nav-link" %>
</li>
<li id="compact-secondary-nav" class="dropdown nav-item">
- <a class="dropdown-toggle nav-link" data-toggle="dropdown" href="#"><%= t "layouts.more" %></a>
+ <a class="dropdown-toggle nav-link" data-bs-toggle="dropdown" href="#"><%= t "layouts.more" %></a>
<ul class="dropdown-menu">
<% if Settings.status != "database_offline" && can?(:index, Issue) %>
<li class="<%= current_page_class(issues_path) %>">
</ul>
<% if current_user && current_user.id %>
<div class='d-inline-flex dropdown user-menu logged-in clearfix'>
- <a class='dropdown-toggle btn btn-outline-secondary pl-2 py-1 flex-grow-1' data-toggle='dropdown' href="#">
- <%= user_thumbnail_tiny(current_user, :width => 25, :height => 25, :class => "user_thumbnail_tiny rounded-sm") %>
+ <a class='dropdown-toggle btn btn-outline-secondary px-2 py-1 flex-grow-1' data-bs-toggle='dropdown' href="#">
+ <%= user_thumbnail_tiny(current_user, :width => 25, :height => 25, :class => "user_thumbnail_tiny rounded-1") %>
<%= render :partial => "layouts/inbox" %>
<span class="user-button">
<span class='username'>
</span>
</span>
</a>
- <div class='dropdown-menu dropdown-menu-right'>
+ <div class='dropdown-menu dropdown-menu-end'>
<%= link_to t("users.show.my_dashboard"), dashboard_path, :class => "dropdown-item" %>
<%= link_to inbox_messages_path, :class => "dropdown-item" do %>
<%= t("users.show.my messages") %>
<div class="search_forms">
<form method="GET" action="<%= search_path %>" class="search_form px-1 py-2">
- <div class="form-row mx-0">
+ <div class="row gx-2 mx-0">
<div class="col">
<div class="input-group flex-nowrap">
<div class='query_wrapper position-relative flex-grow-1'>
<form method="GET" action="<%= directions_path %>" class="directions_form pb-3">
<div class="clearfix px-3 py-3"><span class="icon close"></span></div>
- <div class="form-row m-1">
+ <div class="row gx-2 m-1">
<div class="col-1">
<%= image_tag "marker-green.png", :class => "routing_marker mx-auto d-block", :data => { :type => "from" }, :draggable => "true" %>
</div>
<%= text_field_tag "route_from", params[:from], :placeholder => t("site.search.from"), :autocomplete => "on", :class => "form-control form-control-sm", :dir => "auto" %>
</div>
</div>
- <div class="form-row m-1">
+ <div class="row gx-2 m-1">
<div class="col-1">
<%= image_tag "marker-red.png", :class => "routing_marker mx-auto d-block", :data => { :type => "to" }, :draggable => "true" %>
</div>
<%= text_field_tag "route_to", params[:to], :placeholder => t("site.search.to"), :autocomplete => "on", :class => "form-control form-control-sm", :dir => "auto" %>
</div>
</div>
- <div class="form-row m-1">
+ <div class="row gx-2 m-1">
<div class="col offset-1">
- <select class="routing_engines form-control form-control-sm" name="routing_engines"></select>
+ <select class="routing_engines form-select form-select-sm" name="routing_engines"></select>
</div>
<div class="col-auto">
<%= submit_tag t("site.search.submit_text"), :class => "routing_go btn btn-sm btn-primary", :data => { :disable_with => false } %>
</div>
</div>
- <div class="form-row m-1">
+ <div class="row gx-2 m-1">
<div class="col offset-1">
<a class="reverse_directions"><%= t("site.search.reverse_directions_text") %></a>
</div>
<% end -%>
</ul>
-<div class="row justify-content-start no-gutters mx-n1">
+<div class="row justify-content-start g-0 mx-n1">
<div class="col-auto mx-1">
<%= bootstrap_form_tag :action => :create do |f| %>
<%= f.hidden_field :client_id, :value => @pre_auth.client.uid %>
<% end -%>
</ul>
</td>
- <td class="align-middle text-right">
+ <td class="align-middle text-end">
<%= link_to t(".revoke"), oauth_authorized_application_path(application), { :method => :delete, :class => "btn btn-outline-danger", :data => { :confirm => t(".confirm_revoke") } } %>
</td>
</tr>
<%= f.text_field :url %>
<%= f.text_field :callback_url %>
<%= f.text_field :support_url %>
-<div class='form-group'>
+<div class='mb-3'>
<p><%= t ".requests" %></p>
<% ClientApplication.all_permissions.each do |perm| %>
<%= f.check_box perm %>
<%= bootstrap_form_for current_user, :url => { :action => :update }, :html => { :multipart => true, :autocomplete => :off } do |f| %>
<%= f.richtext_field :description, :cols => 80, :rows => 20 %>
- <fieldset class="form-group">
+ <fieldset class="mb-3">
<%= f.label t(".image") %>
- <div class="form-row">
+ <div class="row">
<div class="col-sm-2">
<%= user_image current_user %>
</div>
<legend><%= t ".home location" -%></legend>
<div id="homerow" <% unless current_user.home_lat and current_user.home_lon %> class="nohome"<% end %>>
<p class="message text-muted"><%= t ".no home location" %></p>
- <div class="form-row">
+ <div class="row">
<%= f.text_field :home_lat, :wrapper_class => "col-sm-4", :id => "home_lat" %>
<%= f.text_field :home_lon, :wrapper_class => "col-sm-4", :id => "home_lon" %>
</div>
<hr>
<div id="loginForm">
- <div class="form-group">
- <label><%= t ".with external" %></label>
+ <div class="mb-3">
+ <label class="form-label"><%= t ".with external" %></label>
<ul class='list-unstyled' id="login_auth_buttons">
<li><%= link_to image_tag("openid.png", :alt => t(".auth_providers.openid.title")), "#", :id => "openid_open_url", :title => t(".auth_providers.openid.title") %></li>
</ul>
<%= form_tag(auth_path(:provider => "openid"), :id => "openid_login_form") do %>
- <div id='login_openid_url' class="form-group">
- <label for='openid_url'><%= t ".openid_html", :logo => openid_logo %></label>
+ <div id='login_openid_url' class="mb-3">
+ <label for='openid_url' class="form-label"><%= t ".openid_html", :logo => openid_logo %></label>
<%= hidden_field_tag("referer", params[:referer], :autocomplete => "off") %>
<%= text_field_tag("openid_url", "", :tabindex => 3, :autocomplete => "on", :class => "openid_url form-control") %>
<span class="form-text text-muted">(<a href="<%= t "accounts.edit.openid.link" %>" target="_new"><%= t "accounts.edit.openid.link text" %></a>)</span>
-<div id="<%= id %>_container" class="form-row richtext_container">
+<div id="<%= id %>_container" class="row richtext_container">
<div id="<%= id %>_content" class="col-sm-8 mb-3 mb-sm-0 richtext_content">
<%= builder.text_area(attribute, options.merge(:wrapper => false, "data-preview-url" => preview_url(:type => type))) %>
<div id="<%= id %>_preview" class="richtext_preview richtext text-break"></div>
</div>
<div id="export_commit">
- <div class="form-group d-flex">
+ <div class="mb-3 d-flex">
<%= submit_tag t(".export_button"), :class => "btn btn-primary mx-auto" %>
</div>
</div>
<% sites = %w[beginners_guide help mailing_lists forums irc switch2osm welcomemat wiki] %>
<% sites.prepend("welcome") if current_user %>
-<% sites.in_groups_of(3, false) do |group| %>
- <div class="card-deck mb-4">
- <% group.each do |site| %>
- <div class='<%= site %> help-item card'>
+<div class="row row-cols-sm-3 g-4 mb-3">
+ <% sites.each do |site| %>
+ <div class="col">
+ <div class='<%= site %> help-item card h-100'>
<div class='card-body'>
<h6 class='card-title'>
<a href='<%= t ".#{site}.url" %>'>
</small>
</div>
</div>
- <% end %>
- </div>
-<% end %>
+ </div>
+ <% end %>
+</div>
<% content_for :heading_class, "pb-0" %>
<% content_for :heading do %>
<h1><%= @title %></h1>
- <p>
- <%= t(".description") %>
- <% if params[:tag] %>
- <%= link_to t(".remove_tag_filter", :tag => params[:tag]), { :controller => "traces", :action => "index", :display_name => nil, :tag => nil, :page => nil }, { :class => "border-left ml-2 pl-2" } %>
- <% end %>
- </p>
+ <nav class="secondary-actions mb-3">
+ <ul>
+ <li>
+ <%= t(".description") %>
+ </li>
+ <% if params[:tag] %>
+ <li>
+ <%= link_to t(".remove_tag_filter", :tag => params[:tag]), { :controller => "traces", :action => "index", :display_name => nil, :tag => nil, :page => nil } %>
+ </li>
+ <% end %>
+ </ul>
+ </nav>
<ul class="nav nav-tabs flex-column flex-sm-row">
<% if @target_user.blank? %>
<!-- public traces -->
<li class="nav-item flex-sm-grow-1"></li>
<li class="nav-item">
- <div class="px-3 py-1">
+ <div class="px-3 py-2 py-sm-0">
<%= link_to({ :action => :georss, :display_name => @target_user&.display_name, :tag => params[:tag] }, { :class => "btn btn-secondary btn-sm" }) do %>
<svg xmlns="http://www.w3.org/2000/svg" width="18" height="18" fill="white" viewBox="0 0 16 16">
<path d="M5.5 12a1.5 1.5 0 1 1-3 0 1.5 1.5 0 0 1 3 0zm-3-8.5a1 1 0 0 1 1-1c5.523 0 10 4.477 10 10a1 1 0 1 1-2 0 8 8 0 0 0-8-8 1 1 0 0 1-1-1zm0 4a1 1 0 0 1 1-1 6 6 0 0 1 6 6 1 1 0 1 1-2 0 4 4 0 0 0-4-4 1 1 0 0 1-1-1z" />
<%= f.richtext_field :reason, :cols => 80, :rows => 20, :format => @user_block.reason_format %>
<%= f.form_group do %>
- <%= label_tag "user_block_period", t(".period") %><br />
- <%= select_tag("user_block_period", options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period])) %>
+ <%= label_tag "user_block_period", t(".period"), :class => "form-label" %>
+ <%= select_tag("user_block_period",
+ options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
+ :class => "form-select") %>
<% end %>
<%= f.form_group :needs_view do %>
<%= f.richtext_field :reason, :cols => 80, :rows => 20 %>
<%= f.form_group do %>
- <%= label_tag "user_block_period", t(".period") %><br />
- <%= select_tag("user_block_period", options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period])) %>
+ <%= label_tag "user_block_period", t(".period"), :class => "form-label" %>
+ <%= select_tag("user_block_period",
+ options_for_select(UserBlock::PERIODS.collect { |h| [block_duration_in_words(h.hours), h.to_s] }, params[:user_block_period]),
+ :class => "form-select") %>
<% end %>
<%= f.form_group :needs_view do %>
</p>
<%= bootstrap_form_for :revoke, :url => { :action => "revoke" } do |f| %>
- <div class="form-group">
+ <div class="mb-3">
<div class="form-check">
<%= check_box_tag "confirm", "yes", false, { :class => "form-check-input" } %>
<%= label_tag "confirm", t(".confirm"), { :class => "form-check-label" } %>
<%= f.text_field :display_name, :help => t(".display name description"), :tabindex => 3 %>
- <fieldset class="form-group" id="auth_field">
- <label for="user_auth_provider"><%= t(".external auth") %></label>
- <div class="form-row">
+ <fieldset class="mb-3" id="auth_field">
+ <label for="user_auth_provider" class="form-label"><%= t(".external auth") %></label>
+ <div class="row">
<%= f.select(:auth_provider, Auth.providers, :default => "", :hide_label => true, :wrapper => { :class => "col-auto mb-0" }, :tabindex => 4) %>
<%= f.text_field(:auth_uid, :hide_label => true, :wrapper => { :class => "col mb-0" }, :tabindex => 5) %>
</div>
<%= t ".heading_ct" %>
</h4>
<p class="text-muted"><%= t ".contributor_terms_explain" %></p>
- <label>
+ <label class="form-label">
<%= t ".legale_select" %>
</label>
- <div class="form-group">
+ <div class="mb-3">
<% [%w[france FR], %w[italy IT], %w[rest_of_world GB]].each do |name, legale| %>
<div class="form-check form-check-inline">
<%= radio_button_tag "legale", legale, @legale == legale, :data => { :url => url_for(:legale => legale) }, :class => "form-check-input" %>
:translations => "https://www.osmfoundation.org/wiki/License/Contributor_Terms/Informal_Translations" %>
</p>
</div>
- <div class="form-group">
+ <div class="mb-3">
<div class="form-check">
<%= check_box_tag "read_ct", "1", false, :class => "form-check-input" %>
<label for="read_ct" class="form-check-label">
</h4>
<p class="text-muted"><%= t ".tou_explain_html", :tou_link => link_to(t("layouts.tou"), "https://wiki.osmfoundation.org/wiki/Terms_of_Use", :target => :new) %></p>
- <div class="form-group">
+ <div class="mb-3">
<div class="form-check">
<%= check_box_tag "read_tou", "1", false, :class => "form-check-input" %>
<label for="read_tou" class="form-check-label">
<%= hidden_field_tag("referer", h(params[:referer])) unless params[:referer].nil? %>
- <div class="form-group">
+ <div class="mb-3">
<%= submit_tag("Continue", :name => "continue", :id => "continue", :disabled => true, :class => "btn btn-primary") %>
<%= submit_tag("Cancel", :name => "decline", :id => "decline", :class => "btn btn-outline-secondary") %>
</div>
- <div class="form-group">
+ <div class="mb-3">
<div class="form-check">
<%= check_box("user", "consider_pd", :class => "form-check-input") %>
<label for="user_consider_pd" class="form-check-label">
assert_response :success
assert_template :edit
assert_select ".notice", false
- assert_select "form#accountForm > div.form-group > input.is-invalid#user_display_name"
+ assert_select "form#accountForm > div > input.is-invalid#user_display_name"
# Changing name to one that exists should fail, regardless of case
new_attributes = user.attributes.dup.merge(:display_name => create(:user).display_name.upcase)
assert_response :success
assert_template :edit
assert_select ".notice", false
- assert_select "form#accountForm > div.form-group > input.is-invalid#user_display_name"
+ assert_select "form#accountForm > div > input.is-invalid#user_display_name"
# Changing name to one that doesn't exist should work
new_attributes = user.attributes.dup.merge(:display_name => "new tester")
assert_response :success
assert_template :edit
assert_select ".notice", /^User information updated successfully/
- assert_select "form#accountForm > div.form-group > input#user_display_name[value=?]", "new tester"
+ assert_select "form#accountForm > div > input#user_display_name[value=?]", "new tester"
# Record the change of name
user.display_name = "new tester"
assert_response :success
assert_template :edit
assert_select ".notice", false
- assert_select "form#accountForm > div.form-group > input.is-invalid#user_new_email"
+ assert_select "form#accountForm > div > input.is-invalid#user_new_email"
# Changing email to one that exists should fail, regardless of case
user.new_email = create(:user).email.upcase
assert_response :success
assert_template :edit
assert_select ".notice", false
- assert_select "form#accountForm > div.form-group > input.is-invalid#user_new_email"
+ assert_select "form#accountForm > div > input.is-invalid#user_new_email"
# Changing email to one that doesn't exist should work
user.new_email = "new_tester@example.com"
assert_response :success
assert_template :edit
assert_select ".notice", /^User information updated successfully/
- assert_select "form#accountForm > div.form-group > input#user_new_email[value=?]", user.new_email
+ assert_select "form#accountForm > div > input#user_new_email[value=?]", user.new_email
email = ActionMailer::Base.deliveries.first
assert_equal 1, email.to.count
assert_equal user.new_email, email.to.first
assert_template :edit
assert_select ".notice", false
assert_select ".error", true
- assert_select "form > div.form-group > select#user_preferred_editor > option[selected]", false
+ assert_select "form > div > select#user_preferred_editor > option[selected]", false
# Changing to a valid editor should work
user.preferred_editor = "id"
assert_template :show
assert_select ".notice", /^Profile updated./
get edit_profile_path
- assert_select "form > fieldset.form-group > div.form-row > div.col-sm-10 > div.form-check > input[name=avatar_action][checked][value=?]", "keep"
+ assert_select "form > fieldset > div > div.col-sm-10 > div.form-check > input[name=avatar_action][checked][value=?]", "keep"
# Changing to a gravatar image should work
put profile_path, :params => { :avatar_action => "gravatar", :user => { :description => user.description } }
assert_template :show
assert_select ".notice", /^Profile updated./
get edit_profile_path
- assert_select "form > fieldset.form-group > div.form-row > div.col-sm-10 > div.form-group > div.form-check > input[name=avatar_action][checked][value=?]", "gravatar"
+ assert_select "form > fieldset > div > div.col-sm-10 > div > div.form-check > input[name=avatar_action][checked][value=?]", "gravatar"
# Removing the image should work
put profile_path, :params => { :avatar_action => "delete", :user => { :description => user.description } }
assert_template :show
assert_select ".notice", /^Profile updated./
get edit_profile_path
- assert_select "form > fieldset.form-group > div.form-row > div.col-sm-10 > div.form-check > input[name=avatar_action][checked]", false
- assert_select "form > fieldset.form-group > div.form-row > div.col-sm-10 > div.form-group > div.form-check > input[name=avatar_action][checked]", false
+ assert_select "form > fieldset > div > div.col-sm-10 > div > input[name=avatar_action][checked]", false
+ assert_select "form > fieldset > div > div.col-sm-10 > div > div.form-check > input[name=avatar_action][checked]", false
end
end
assert_response :success
assert_template "new"
- assert_select "form > div.form-group > input.is-invalid#user_email"
+ assert_select "form > div > input.is-invalid#user_email"
end
def test_save_duplicate_email
assert_response :success
assert_template "new"
- assert_select "form > div.form-group > input.is-invalid#user_email"
+ assert_select "form > div > input.is-invalid#user_email"
end
def test_save_duplicate_email_uppercase
assert_response :success
assert_template "new"
- assert_select "form > div.form-group > input.is-invalid#user_email"
+ assert_select "form > div > input.is-invalid#user_email"
end
def test_save_duplicate_name
assert_response :success
assert_template "new"
- assert_select "form > div.form-group > input.is-invalid#user_display_name"
+ assert_select "form > div > input.is-invalid#user_display_name"
end
def test_save_duplicate_name_uppercase
assert_response :success
assert_template "new"
- assert_select "form > div.form-group > input.is-invalid#user_display_name"
+ assert_select "form > div > input.is-invalid#user_display_name"
end
def test_save_blocked_domain
assert_template "users/new"
assert_equal locale.to_s, response.headers["Content-Language"]
assert_select "form"
- assert_select "form > div.form-group > input.is-invalid#user_email"
+ assert_select "form > div > input.is-invalid#user_email"
assert_no_missing_translations
end
end
end
assert_response :success
assert_template "users/new"
- assert_select "form > div.form-group > input.is-invalid#user_display_name"
+ assert_select "form > div > input.is-invalid#user_display_name"
assert_no_missing_translations
end
end
projects / rails.git / commitdiff