Skip to content

Commit d192c0d

Browse files
joedolsonjoedolson
committed
Clean up; style name changes.
1 parent f480454 commit d192c0d

File tree

3 files changed

+54
-31
lines changed

3 files changed

+54
-31
lines changed

‎src/assets/css/admin.css‎

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -47,18 +47,18 @@
4747
display: block;
4848
}
4949

50-
.mc-tabs .tabs, .mc-tablinks {
50+
.ableplayer-tabs .tabs, .mc-tablinks {
5151
position: relative;
5252
border-bottom: 1px solid #c3c4c7;
5353
margin-bottom: 0;
5454
margin-top: 20px;
5555
}
5656

57-
.mc-tabs.vertical .tabs {
57+
.ableplayer-tabs.vertical .tabs {
5858
border-bottom: none;
5959
}
6060

61-
.mc-tabs .tabs button, .mc-tablinks a {
61+
.ableplayer-tabs .tabs button, .mc-tablinks a {
6262
display: inline-block;
6363
padding: 6px 12px;
6464
font-size: 14px;
@@ -69,7 +69,7 @@
6969
border-bottom: 2px solid transparent;
7070
}
7171

72-
.mc-tabs .tabs button[aria-selected=true]:after,
72+
.ableplayer-tabs .tabs button[aria-selected=true]:after,
7373
.mc-tablinks a[aria-current=page]:after {
7474
content: '';
7575
position: absolute;
@@ -83,7 +83,7 @@
8383
clear: both;
8484
}
8585

86-
.mc-tabs .tabs button[aria-selected=true]:before,
86+
.ableplayer-tabs .tabs button[aria-selected=true]:before,
8787
.mc-tablinks a[aria-current=page]:before {
8888
content: '';
8989
position: absolute;
@@ -98,8 +98,8 @@
9898
}
9999

100100
@media screen and ( width >= 481px ) {
101-
.mc-tabs.vertical .tabs button[aria-selected=true]:after,
102-
.mc-tabs.vertical .tabs button[aria-selected=true]:before {
101+
.ableplayer-tabs.vertical .tabs button[aria-selected=true]:after,
102+
.ableplayer-tabs.vertical .tabs button[aria-selected=true]:before {
103103
left: 100%;
104104
top: 50%;
105105
border-left: 10px solid #fff;
@@ -108,7 +108,7 @@
108108
margin-top: -10px;
109109
}
110110

111-
.mc-tabs.vertical .tabs button[aria-selected=true]:before {
111+
.ableplayer-tabs.vertical .tabs button[aria-selected=true]:before {
112112
left: calc( 100% + 1px );
113113
top: 50%;
114114
border-left: 10px solid #a2a4a7;
@@ -118,28 +118,28 @@
118118
}
119119

120120

121-
.mc-tabs .tabs button:hover,
122-
.mc-tabs .tabs button:focus,
121+
.ableplayer-tabs .tabs button:hover,
122+
.ableplayer-tabs .tabs button:focus,
123123
.mc-tablinks a:hover,
124124
.mc-tablinks a:focus {
125125
border-bottom: 2px solid #000;
126126
}
127127

128-
.mc-tabs .tabs button[aria-selected=true],
128+
.ableplayer-tabs .tabs button[aria-selected=true],
129129
.mc-tablinks a[aria-current=page] {
130130
background: #fff;
131131
}
132132

133-
.mc-tabs.vertical {
133+
.ableplayer-tabs.vertical {
134134
display: grid;
135135
grid-template-columns: 12em 1fr;
136136
}
137137

138-
.mc-tabs.vertical .tabs {
138+
.ableplayer-tabs.vertical .tabs {
139139
margin-top: 0;
140140
}
141141

142-
.mc-tabs.vertical .tabs button {
142+
.ableplayer-tabs.vertical .tabs button {
143143
display: block;
144144
border: 1px solid #c3c4c7;
145145
border-right: none;
@@ -149,36 +149,36 @@
149149
text-align: left;
150150
}
151151

152-
.mc-tabs.vertical .tabs button:last-of-type {
152+
.ableplayer-tabs.vertical .tabs button:last-of-type {
153153
border-bottom: 1px solid #c3c4c7;
154154
}
155155

156-
.mc-tabs.vertical .wptab {
156+
.ableplayer-tabs.vertical .wptab {
157157
border-left: 1px solid #c3c4c7;
158158
padding: 10px;
159159
}
160160

161161
@media screen and ( width < 481px ) {
162-
.mc-tabs.vertical {
162+
.ableplayer-tabs.vertical {
163163
display: flex;
164164
flex-wrap: wrap;
165165
}
166-
.mc-tabs.vertical .tabs {
166+
.ableplayer-tabs.vertical .tabs {
167167
display: flex;
168168
flex-wrap: wrap;
169169
}
170-
.mc-tabs.vertical .tabs button {
170+
.ableplayer-tabs.vertical .tabs button {
171171
border: 1px solid #c3c4c7;
172172
}
173-
.mc-tabs.vertical .wptab {
173+
.ableplayer-tabs.vertical .wptab {
174174
padding: 0;
175175
margin-top: 10px;
176176
}
177-
.mc-tabs.vertical .tabs button[aria-selected="true"]::before,
178-
.mc-tabs.vertical .tabs button[aria-selected="true"]::after {
177+
.ableplayer-tabs.vertical .tabs button[aria-selected="true"]::before,
178+
.ableplayer-tabs.vertical .tabs button[aria-selected="true"]::after {
179179
display: none;
180180
}
181-
.mc-tabs.vertical .tabs button[aria-selected="true"] {
181+
.ableplayer-tabs.vertical .tabs button[aria-selected="true"] {
182182
border-bottom: 2px solid black;
183183
}
184184
}

‎src/assets/js/admin.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
jQuery(document).ready(function ($) {
22
let firstItem = window.location.hash;
3-
const tabGroups = document.querySelectorAll( '.mc-tabs' );
3+
const tabGroups = document.querySelectorAll( '.ableplayer-tabs' );
44
for ( let i = 0; i < tabGroups.length; i++ ) {
55
const panel = $( tabGroups[i] ).find( firstItem );
66
if ( panel.length !== 0 ) {
@@ -10,7 +10,7 @@ jQuery(document).ready(function ($) {
1010
showPanel( '#' + firstItem );
1111
}
1212
}
13-
const tabs = document.querySelectorAll('.mc-tabs [role=tab]'); // get all role=tab elements as a variable.
13+
const tabs = document.querySelectorAll('.ableplayer-tabs [role=tab]'); // get all role=tab elements as a variable.
1414
for ( let i = 0; i < tabs.length; i++) {
1515
tabs[i].addEventListener( 'click', showTabPanel );
1616
tabs[i].addEventListener( 'keydown', handleKeyPress );
@@ -23,7 +23,7 @@ jQuery(document).ready(function ($) {
2323
function showTabPanel(e) {
2424
const tabContainer = $( e.currentTarget ).closest( '.tabs' );
2525
const tabs = tabContainer.find( '[role=tab]' );
26-
const container = $( e.currentTarget ).closest( '.mc-tabs' );
26+
const container = $( e.currentTarget ).closest( '.ableplayer-tabs' );
2727
const inside = $( e.currentTarget ).parents( '.inside' );
2828
const tabPanelToOpen = e.target.getAttribute('aria-controls');
2929
const iframes = $( 'iframe.mc-iframe' );
@@ -63,7 +63,7 @@ jQuery(document).ready(function ($) {
6363
const control = $( 'button[aria-controls=' + id + ']' );
6464
const tabContainer = $( hash ).closest( '.tabs' );
6565
const tabs = tabContainer.find( '[role=tab]' );
66-
const container = $( hash ).closest( '.mc-tabs' );
66+
const container = $( hash ).closest( '.ableplayer-tabs' );
6767
const tabPanels = container.find( '[role=tabpanel]' );
6868
const currentPanel = document.getElementById(id);
6969

‎src/inc/settings.php‎

Lines changed: 27 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function ableplayer_update_setting( $key, $value = '' ) {
4141
$settings = get_option( 'ableplayer_settings', ableplayer_default_settings() );
4242

4343
$settings[ $key ] = $value;
44-
$return = update_option( 'ableplayer_settings', $settings );
44+
$return = update_option( 'ableplayer_settings', $settings );
4545

4646
return $return;
4747
}
@@ -224,14 +224,33 @@ function ableplayer_settings_field( $args = array() ) {
224224
}
225225
}
226226

227+
/**
228+
* Save a group of AblePlayer settings.
229+
*
230+
* @param array $settings An array of settings.
231+
*
232+
* @return bool
233+
*/
234+
function ableplayer_update_options( $settings ) {
235+
if ( empty( $settings ) ) {
236+
return false;
237+
}
238+
$options = get_option( 'ableplayer_settings' );
239+
$settings = array_merge( $options, $settings );
240+
241+
return update_option( 'ableplayer_settings', $settings );
242+
}
227243

228244
/**
229245
* Update AblePlayer settings.
230246
*
231247
* @param array $post POST data.
232248
*/
233249
function ableplayer_update_settings( $post ) {
250+
$setting = ( ! empty( $post['example'] ) && 'on' === $post['example'] ) ? 'true' : 'false';
251+
$settings['api_enabled'] = $setting;
234252

253+
ableplayer_update_options( $settings );
235254
}
236255

237256
/**
@@ -257,7 +276,7 @@ function ableplayer_settings_form() {
257276
?>
258277
<div class="wrap ableplayer-admin ableplayer-settings-page" id="ableplayer_settings">
259278
<h1><?php esc_html_e( 'My Calendar Settings', 'ableplayer' ); ?></h1>
260-
<div class="mc-tabs">
279+
<div class="ableplayer-tabs">
261280
<div class="tabs" role="tablist" data-default="ableplayer-settings">
262281
<button type="button" role="tab" aria-selected="false" id="tab_settings" aria-controls="ableplayer-settings"><?php esc_html_e( 'Settings', 'ableplayer' ); ?></button>
263282
<button type="button" role="tab" aria-selected="false" id="tab_shortcode" aria-controls="ableplayer-shortcode"><?php esc_html_e( 'Shortcodes', 'ableplayer' ); ?></button>
@@ -328,8 +347,12 @@ function ableplayer_show_sidebar() {
328347

329348
<div class="inside">
330349
<p>
331-
<?php echo wp_kses_post( sprintf( __( 'Learn more about the <a href="%s">AblePlayer accessible media player</a>.', 'ableplayer' ), 'https://ableplayer.github.io/ableplayer/' ) ); ?>
332-
<?php echo wp_kses_post( sprintf( __( '<a href="%s">Support Joe Dolson</a>, AblePlayer lead developer.', 'ableplayer' ), 'https://www.joedolson.com/donate/' ) ); ?>
350+
<?php
351+
// Translators: URL for AblePlayer github docs.
352+
echo wp_kses_post( sprintf( __( 'Learn more about the <a href="%s">AblePlayer accessible media player</a>.', 'ableplayer' ), 'https://ableplayer.github.io/ableplayer/' ) );
353+
// Translators: URL for Joe Dolson donate page.
354+
echo wp_kses_post( sprintf( __( '<a href="%s">Support Joe Dolson</a>, AblePlayer lead developer.', 'ableplayer' ), 'https://www.joedolson.com/donate/' ) );
355+
?>
333356
</p>
334357
<ul class="ableplayer-flex ableplayer-social">
335358
<li><a href="https://toot.io/@joedolson">

0 commit comments

Comments
 (0)