-
Notifications
You must be signed in to change notification settings - Fork 205
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.
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
Module/form manager #2098
base: develop
Are you sure you want to change the base?
Module/form manager #2098
Changes from 124 commits
f0d6e79
2dbcde1
bedbe32
440dac1
a55be72
bc789b4
18057d7
c391f7b
2f430c9
3eb324c
9ada0e4
7b36504
50402b0
d17cb3d
d8edffa
6f0e7f7
9d90ea6
63d0b60
af756c6
a634761
b869ff8
2e410e6
fc5ff41
62aa976
70553c5
bb27153
7f2f9fb
dc0d791
870738d
7475c22
7961933
f1890fa
ecdf1c4
00fe573
aee9ca2
04239aa
19731cb
400cc06
b41f4ea
0746cd1
0254f10
08090af
39ffc81
8945cb8
2598b21
a8491c0
889449d
c27a0eb
afa7080
e13028e
acb81d2
581a09c
69b8a12
645a7ba
1e399eb
0198461
633b521
a743cd1
929f5bc
70e2ea0
58cbdba
239f504
fa3fb9b
8b66e38
1376cc7
c4cbc84
c96ee57
dd45805
8d666bd
d4563c6
c72fef6
f1f48a9
ee58880
d45b8e7
f7ce607
bf36c8d
a434e13
88cc82b
229139f
9e08b18
5750393
daaaa26
2c42450
ec88e80
531bc8b
985684f
0c7d55f
64e16f2
599d413
34029c7
8082724
e6ea576
cb51e17
aaee6a1
8ee16c2
693c6de
374c796
3c175da
654614a
59aa377
659e645
02adcbf
6043469
935f03e
7c0c718
40b08c6
49825ee
e2ef682
4b33118
dd6f3db
0e05b17
78eaa59
fabbc1a
ec05bc8
cd745e8
36f4fc2
faae548
38556db
1c91942
b92694d
cd4c744
2af1fbb
b13cd3b
a04c74b
0ecb790
1c36c1c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,7 @@ | |
* @property WeDevs\Dokan\BackgroundProcess\Manager $bg_process Instance of WeDevs\Dokan\BackgroundProcess\Manager class | ||
* @property WeDevs\Dokan\Withdraw\Manager $withdraw Instance of WeDevs\Dokan\Withdraw\Manager class | ||
* @property WeDevs\Dokan\Frontend\Frontend $frontend_manager Instance of \WeDevs\Dokan\Frontend\Frontend class | ||
* @property WeDevs\Dokan\Dashboard\Manager $dashboard Instance of \WeDevs\Dokan\Dashboard\Manager class | ||
* @property WeDevs\Dokan\Registration $registration Instance of WeDevs\Dokan\Registration class | ||
*/ | ||
final class WeDevs_Dokan { | ||
|
@@ -214,6 +215,7 @@ public function define_constants() { | |
defined( 'DOKAN_PLUGIN_VERSION' ) || define( 'DOKAN_PLUGIN_VERSION', $this->version ); | ||
defined( 'DOKAN_DIR' ) || define( 'DOKAN_DIR', __DIR__ ); | ||
defined( 'DOKAN_INC_DIR' ) || define( 'DOKAN_INC_DIR', __DIR__ . '/includes' ); | ||
defined( 'DOKAN_TEMPLATE_DIR' ) || define( 'DOKAN_TEMPLATE_DIR', __DIR__ . '/templates' ); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Codebase verification Template directory constant needs to be used consistently The verification reveals inconsistent template path handling across the codebase:
For consistency and maintainability:
🔗 Analysis chainVerify template directory constant usage The constant definition looks good, but let's ensure it's being used consistently across the codebase. 🏁 Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Description: Check for existing usages of hardcoded template paths that should use DOKAN_TEMPLATE_DIR
# Search for hardcoded template paths
rg -g '*.php' "(?i)(templates?.*directory|template.*path|template.*dir)"
# Search for current usage of the new constant
rg "DOKAN_TEMPLATE_DIR"
Length of output: 6453 |
||
defined( 'DOKAN_LIB_DIR' ) || define( 'DOKAN_LIB_DIR', __DIR__ . '/lib' ); | ||
defined( 'DOKAN_PLUGIN_ASSEST' ) || define( 'DOKAN_PLUGIN_ASSEST', plugins_url( 'assets', DOKAN_FILE ) ); | ||
|
||
|
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -6,6 +6,7 @@ | |||||||||||||||||||||||||||||||||
use WeDevs\Dokan\ReverseWithdrawal\SettingsHelper; | ||||||||||||||||||||||||||||||||||
use WeDevs\Dokan\ProductCategory\Helper as CategoryHelper; | ||||||||||||||||||||||||||||||||||
use WeDevs\Dokan\Utilities\OrderUtil; | ||||||||||||||||||||||||||||||||||
use WeDevs\Dokan\ProductForm\Factory as ProductFormFactory; | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
class Assets { | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
|
@@ -527,7 +528,7 @@ public function get_scripts() { | |||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
'dokan-util-helper' => [ | ||||||||||||||||||||||||||||||||||
'src' => $asset_url . '/js/helper.js', | ||||||||||||||||||||||||||||||||||
'deps' => [ 'jquery', 'dokan-sweetalert2', 'moment' ], | ||||||||||||||||||||||||||||||||||
'deps' => [ 'jquery', 'dokan-sweetalert2', 'moment', 'jquery-tiptip' ], | ||||||||||||||||||||||||||||||||||
'version' => filemtime( $asset_path . 'js/helper.js' ), | ||||||||||||||||||||||||||||||||||
'in_footer' => false, | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
|
@@ -572,6 +573,15 @@ public function get_scripts() { | |||||||||||||||||||||||||||||||||
return $scripts; | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
public function get_product_fields() { | ||||||||||||||||||||||||||||||||||
$temp_fields = []; | ||||||||||||||||||||||||||||||||||
foreach ( ProductFormFactory::get_fields() as $field_id => $field ) { | ||||||||||||||||||||||||||||||||||
$temp_fields[ $field_id ] = $field->toArray(); | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
return json_encode( $temp_fields ); | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
Comment on lines
+576
to
+583
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use wp_json_encode for WordPress compatibility. The current implementation uses PHP's native Apply this fix: public function get_product_fields() {
$temp_fields = [];
foreach ( ProductFormFactory::get_fields() as $field_id => $field ) {
$temp_fields[ $field_id ] = $field->toArray();
}
- return json_encode( $temp_fields );
+ return wp_json_encode( $temp_fields );
} 📝 Committable suggestion
Suggested change
🧰 Tools🪛 GitHub Check: Run PHPCS inspection[warning] 582-582: |
||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
/** | ||||||||||||||||||||||||||||||||||
* Enqueue front-end scripts | ||||||||||||||||||||||||||||||||||
*/ | ||||||||||||||||||||||||||||||||||
|
@@ -617,6 +627,7 @@ public function enqueue_front_scripts() { | |||||||||||||||||||||||||||||||||
'currency_format' => esc_attr( str_replace( [ '%1$s', '%2$s' ], [ '%s', '%v' ], get_woocommerce_price_format() ) ), // For accounting JS | ||||||||||||||||||||||||||||||||||
'round_at_subtotal' => get_option( 'woocommerce_tax_round_at_subtotal', 'no' ), | ||||||||||||||||||||||||||||||||||
'product_types' => apply_filters( 'dokan_product_types', [ 'simple' ] ), | ||||||||||||||||||||||||||||||||||
'product_form_fields' => $this->get_product_fields(), | ||||||||||||||||||||||||||||||||||
'loading_img' => DOKAN_PLUGIN_ASSEST . '/images/loading.gif', | ||||||||||||||||||||||||||||||||||
'store_product_search_nonce' => wp_create_nonce( 'dokan_store_product_search_nonce' ), | ||||||||||||||||||||||||||||||||||
'i18n_download_permission' => __( 'Are you sure you want to revoke access to this download?', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
|
@@ -792,11 +803,9 @@ public function load_dokan_global_scripts() { | |||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
'sweetalert_local' => [ | ||||||||||||||||||||||||||||||||||
'cancelButtonText' => __( 'Cancel', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'closeButtonText' => __( 'Close', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'confirmButtonText' => __( 'OK', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'denyButtonText' => __( 'No', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'closeButtonAriaLabel' => __( 'Close this dialog', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'confirmButtonText' => __( 'OK', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'denyButtonText' => __( 'No', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'cancelButtonText' => __( 'Cancel', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
], | ||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||
|
@@ -988,43 +997,21 @@ public function conditional_localized_args( $default_args ) { | |||||||||||||||||||||||||||||||||
$has_flex_height = ! empty( $general_settings['store_banner_flex_height'] ) ? $general_settings['store_banner_flex_height'] : true; | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
$custom_args = [ | ||||||||||||||||||||||||||||||||||
'i18n_choose_featured_img' => __( 'Upload featured image', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_file' => __( 'Choose a file', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_gallery' => __( 'Add Images to Product Gallery', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_featured_img_btn_text' => __( 'Set featured image', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_file_btn_text' => __( 'Insert file URL', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_gallery_btn_text' => __( 'Add to gallery', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'duplicates_attribute_messg' => __( 'Sorry, this attribute option already exists, Try a different one.', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'variation_unset_warning' => __( 'Warning! This product will not have any variations if this option is not checked.', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'new_attribute_prompt' => __( 'Enter a name for the new attribute term:', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'remove_attribute' => __( 'Remove this attribute?', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'dokan_placeholder_img_src' => wc_placeholder_img_src(), | ||||||||||||||||||||||||||||||||||
'add_variation_nonce' => wp_create_nonce( 'add-variation' ), | ||||||||||||||||||||||||||||||||||
'link_variation_nonce' => wp_create_nonce( 'link-variations' ), | ||||||||||||||||||||||||||||||||||
'delete_variations_nonce' => wp_create_nonce( 'delete-variations' ), | ||||||||||||||||||||||||||||||||||
'load_variations_nonce' => wp_create_nonce( 'load-variations' ), | ||||||||||||||||||||||||||||||||||
'save_variations_nonce' => wp_create_nonce( 'save-variations' ), | ||||||||||||||||||||||||||||||||||
'bulk_edit_variations_nonce' => wp_create_nonce( 'bulk-edit-variations' ), | ||||||||||||||||||||||||||||||||||
/* translators: %d: max linked variation. */ | ||||||||||||||||||||||||||||||||||
'i18n_link_all_variations' => esc_js( sprintf( __( 'Are you sure you want to link all variations? This will create a new variation for each and every possible combination of variation attributes (max %d per run).', 'dokan-lite' ), defined( 'WC_MAX_LINKED_VARIATIONS' ) ? WC_MAX_LINKED_VARIATIONS : 50 ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_a_value' => esc_js( __( 'Enter a value', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_menu_order' => esc_js( __( 'Variation menu order (determines position in the list of variations)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_a_value_fixed_or_percent' => esc_js( __( 'Enter a value (fixed or %)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_delete_all_variations' => esc_js( __( 'Are you sure you want to delete all variations? This cannot be undone.', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_last_warning' => esc_js( __( 'Last warning, are you sure?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_image' => esc_js( __( 'Choose an image', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_set_image' => esc_js( __( 'Set variation image', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_added' => esc_js( __( 'variation added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variations_added' => esc_js( __( 'variations added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_no_variations_added' => esc_js( __( 'No variations added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_remove_variation' => esc_js( __( 'Are you sure you want to remove this variation?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_scheduled_sale_start' => esc_js( __( 'Sale start date (YYYY-MM-DD format or leave blank)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_scheduled_sale_end' => esc_js( __( 'Sale end date (YYYY-MM-DD format or leave blank)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_edited_variations' => esc_js( __( 'Save changes before changing page?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_count_single' => esc_js( __( '%qty% variation', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_count_plural' => esc_js( __( '%qty% variations', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_no_result_found' => esc_js( __( 'No Result Found', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_sales_price_error' => esc_js( __( 'Please insert value less than the regular price!', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_featured_img' => __( 'Upload featured image', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_file' => __( 'Choose a file', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_gallery' => __( 'Add Images to Product Gallery', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_featured_img_btn_text' => __( 'Set featured image', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_file_btn_text' => __( 'Insert file URL', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'i18n_choose_gallery_btn_text' => __( 'Add to gallery', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'duplicates_attribute_messg' => __( 'Sorry, this attribute option already exists, Try a different one.', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'variation_unset_warning' => __( 'Warning! This product will not have any variations if this option is not checked.', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'new_attribute_prompt' => __( 'Enter a name for the new attribute term:', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'remove_attribute' => __( 'Remove this attribute?', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'dokan_placeholder_img_src' => wc_placeholder_img_src(), | ||||||||||||||||||||||||||||||||||
'i18n_choose_image' => esc_js( __( 'Choose an image', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_set_image' => esc_js( __( 'Set variation image', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_no_result_found' => esc_js( __( 'No Result Found', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_sales_price_error' => esc_js( __( 'Please insert value less than the regular price!', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
/* translators: %s: decimal */ | ||||||||||||||||||||||||||||||||||
'i18n_decimal_error' => sprintf( __( 'Please enter with one decimal point (%s) without thousand separators.', 'dokan-lite' ), $decimal ), | ||||||||||||||||||||||||||||||||||
/* translators: %s: price decimal separator */ | ||||||||||||||||||||||||||||||||||
|
@@ -1071,6 +1058,30 @@ public function conditional_localized_args( $default_args ) { | |||||||||||||||||||||||||||||||||
'update_settings' => __( 'Update Settings', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
'search_downloadable_products_nonce' => wp_create_nonce( 'search-downloadable-products' ), | ||||||||||||||||||||||||||||||||||
'search_downloadable_products_placeholder' => __( 'Please enter 3 or more characters', 'dokan-lite' ), | ||||||||||||||||||||||||||||||||||
// Todo: Variation related i18n strings will be moved to the Dokan Pro Assets class after releasing product form manager module. | ||||||||||||||||||||||||||||||||||
// Variation related i18n, file: product-variation.js | ||||||||||||||||||||||||||||||||||
'add_variation_nonce' => wp_create_nonce( 'add-variation' ), | ||||||||||||||||||||||||||||||||||
'link_variation_nonce' => wp_create_nonce( 'link-variations' ), | ||||||||||||||||||||||||||||||||||
'delete_variations_nonce' => wp_create_nonce( 'delete-variations' ), | ||||||||||||||||||||||||||||||||||
'load_variations_nonce' => wp_create_nonce( 'load-variations' ), | ||||||||||||||||||||||||||||||||||
'save_variations_nonce' => wp_create_nonce( 'save-variations' ), | ||||||||||||||||||||||||||||||||||
'bulk_edit_variations_nonce' => wp_create_nonce( 'bulk-edit-variations' ), | ||||||||||||||||||||||||||||||||||
/* translators: %d: max linked variation. */ | ||||||||||||||||||||||||||||||||||
'i18n_link_all_variations' => esc_js( sprintf( __( 'Are you sure you want to link all variations? This will create a new variation for each and every possible combination of variation attributes (max %d per run).', 'dokan-lite' ), defined( 'WC_MAX_LINKED_VARIATIONS' ) ? WC_MAX_LINKED_VARIATIONS : 50 ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_a_value' => esc_js( __( 'Enter a value', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_menu_order' => esc_js( __( 'Variation menu order (determines position in the list of variations)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_enter_a_value_fixed_or_percent' => esc_js( __( 'Enter a value (fixed or %)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_delete_all_variations' => esc_js( __( 'Are you sure you want to delete all variations? This cannot be undone.', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_last_warning' => esc_js( __( 'Last warning, are you sure?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_added' => esc_js( __( 'Variation added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variations_added' => esc_js( __( 'Variations added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_no_variations_added' => esc_js( __( 'No variations added', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_remove_variation' => esc_js( __( 'Are you sure you want to remove this variation?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_scheduled_sale_start' => esc_js( __( 'Sale start date (YYYY-MM-DD format or leave blank)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_scheduled_sale_end' => esc_js( __( 'Sale end date (YYYY-MM-DD format or leave blank)', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_edited_variations' => esc_js( __( 'Save changes before changing page?', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_count_single' => esc_js( __( '%qty% variation', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
'i18n_variation_count_plural' => esc_js( __( '%qty% variations', 'dokan-lite' ) ), | ||||||||||||||||||||||||||||||||||
]; | ||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||
$default_args = array_merge( $default_args, $custom_args ); | ||||||||||||||||||||||||||||||||||
|
@@ -1216,6 +1227,7 @@ public function get_admin_localized_scripts() { | |||||||||||||||||||||||||||||||||
'decimal_point' => $decimal, | ||||||||||||||||||||||||||||||||||
'mon_decimal_point' => wc_get_price_decimal_separator(), | ||||||||||||||||||||||||||||||||||
'i18n_date_format' => wc_date_format(), | ||||||||||||||||||||||||||||||||||
'product_types' => apply_filters( 'dokan_product_types', [ 'simple' ] ), | ||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codebase verification
Padding change appears inconsistent with responsive design patterns
The codebase shows multiple declarations of
.instruction-inside
with different padding approaches:The change from percentage-based padding to fixed pixels could cause layout issues across different viewport sizes, especially since the file already contains mobile-specific media queries (@media max-width: 430px).
Recommend:
🔗 Analysis chain
Verify the increased padding impact on layout.
The padding-top has been significantly increased from 20px to 80px. While this might improve visual spacing, please verify that it doesn't cause any layout issues or excessive whitespace in different viewport sizes.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 262
Script:
Length of output: 4309