Skip to content

table widget #2

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

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from 5 commits
Commits
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
1 change: 1 addition & 0 deletions includes/managers/widgets.php
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ private function init_widgets() {
'common',
'inner-section',
'heading',
'table',
'image',
'text-editor',
'video',
Expand Down
288 changes: 288 additions & 0 deletions includes/widgets/table.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,288 @@
<?php
namespace Elementor;

if ( ! defined( 'ABSPATH' ) ) {
exit; // Exit if accessed directly.
}


use Elementor\Core\Kits\Documents\Tabs\Global_Typography;

/**
* Elementor table widget.
*
*
* @since 1.0.0
*/
class Widget_Table extends Widget_Base {

/**
* Get widget name.
*
* Retrieve table widget name.
*
* @since 1.0.0
* @access public
*
* @return string Widget name.
*/
public function get_name() {
return 'table';
}

/**
* Get widget title.
*
* Retrieve table widget title.
*
* @since 1.0.0
* @access public
*
* @return string Widget title.
*/
public function get_title() {
return esc_html__( 'Table', 'elementor' );
}

/**
* Get widget icon.
*
* Retrieve table widget icon.
*
* @since 1.0.0
* @access public
*
* @return string Widget icon.
*/
public function get_icon() {
return 'eicon-table';
}

/**
* Get widget categories.
*
* Retrieve the list of categories the table widget belongs to.
*
* Used to determine where to display the widget in the editor.
*
* @since 2.0.0
* @access public
*
* @return array Widget categories.
*/
public function get_categories() {
return [ 'basic' ];
}

/**
* Get widget keywords.
*
* Retrieve the list of keywords the widget belongs to.
*
* @since 2.1.0
* @access public
*
* @return array Widget keywords.
*/
public function get_keywords() {
return [ 'table'];
}

/**
* Get widget upsale data.
*
* Retrieve the widget promotion data.
*
* @since 3.18.0
* @access protected
*
* @return array Widget promotion data.
*/
protected function get_upsale_data() {
Copy link
Owner

Choose a reason for hiding this comment

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

Delete?

Copy link
Author

@Mchris-hub Mchris-hub Apr 11, 2024

Choose a reason for hiding this comment

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

i deleted it

return [
'condition' => ! Utils::has_pro(),
'image' => esc_url( ELEMENTOR_ASSETS_URL . 'images/go-pro.svg' ),
'image_alt' => esc_attr__( 'Upgrade', 'elementor' ),
'description' => esc_html__( 'Create captivating tables.', 'elementor' ),
'upgrade_url' => esc_url( 'https://go.elementor.com/go-pro-table-widget/' ),
'upgrade_text' => esc_html__( 'Upgrade Now', 'elementor' ),
];
}

/**
* Register table widget controls.
*
* Adds different input fields to allow the user to change and customize the widget settings.
*
* @since 3.1.0
* @access protected
*/
protected function register_controls() {

$this->start_controls_section(
'content_section',
[
'label' => esc_html__( 'Table content', 'elementor' ),
'tab' => \Elementor\Controls_Manager::TAB_CONTENT,
]
);

$this->add_control(
'show_title',
[
'label' => esc_html__( 'Show Columns Titles', 'elementor' ),
'type' => \Elementor\Controls_Manager::SWITCHER,
'label_on' => esc_html__( 'Show', 'elementor' ),
'label_off' => esc_html__( 'Hide', 'elementor' ),
'return_value' => 'yes',
'default' => 'yes',
]
);

$this->add_control(
'column_one',
[
'label' => esc_html__( 'Title column 1', 'elementor' ),
'type' => Controls_Manager::TEXT,
'label_block' => true,
'placeholder' => esc_html__( 'Enter column title', 'elementor' ),
'dynamic' => [
'active' => true,
],
]
);

$this->add_control(
'column_two',
[
'label' => esc_html__( 'Title column 2', 'elementor' ),
'type' => Controls_Manager::TEXT,
'label_block' => true,
'placeholder' => esc_html__( 'Enter column title', 'elementor' ),
'dynamic' => [
'active' => true,
],
]
);
$this->add_control(
'column_three',
[
'label' => esc_html__( 'Title column 3', 'elementor' ),
'type' => Controls_Manager::TEXT,
'label_block' => true,
'placeholder' => esc_html__( 'Enter column title', 'elementor' ),
'dynamic' => [
'active' => true,
],
]
);


$this->add_control(
'rows',
[
'label' => esc_html__( "Rows' contents", 'elementor' ),
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
'label' => esc_html__( "Rows' contents", 'elementor' ),
'label' => esc_html__( 'Content rows', 'elementor' ),

Copy link
Author

Choose a reason for hiding this comment

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

I changed it

'show_label' => esc_html__( true, 'elementor' ),
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
'show_label' => esc_html__( true, 'elementor' ),
'show_label' => true,

Copy link
Author

Choose a reason for hiding this comment

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

I changed it

'type' => \Elementor\Controls_Manager::REPEATER,
'fields' => [
[
'name' => 'column_content_one',
'label' => esc_html__( 'Content column 1', 'elementor' ),
'type' => \Elementor\Controls_Manager::TEXT,
'default' => esc_html__( 'Content column 1' , 'elementor' ),
Copy link
Owner

Choose a reason for hiding this comment

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

Remove default values.

Copy link
Author

Choose a reason for hiding this comment

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

I changed it

'label_block' => false,
],
[
'name' => 'column_content_two',
'label' => esc_html__( 'column content 2', 'elementor' ),
'type' => \Elementor\Controls_Manager::TEXT,
'default' => esc_html__( 'Content column 2' , 'elementor' ),
'label_block' => false,
],
[
'name' => 'column_content_three',
'label' => esc_html__( 'column content 3', 'elementor' ),
'type' => \Elementor\Controls_Manager::TEXT,
'default' => esc_html__( 'Content column 3' , 'elementor' ),
'label_block' => false,
],
],
'title_field' => esc_html__( 'row', 'elementor'),

]
);

$this->end_controls_section();
}

/**
* Render table widget output on the frontend.
*
* Written in PHP and used to generate the final HTML.
*
* @since 1.0.0
* @access protected
*/

protected function render() {
$settings = $this->get_settings_for_display();

echo '<table>';
if ( 'yes' === $settings['show_title'] ) {
echo '<thead>';
echo '<tr>';
echo '<th>' . $settings['column_one'] . '</th>';
echo '<th>' . $settings['column_two'] . '</th>';
echo '<th>' . $settings['column_three'] . '</th>';
echo '</tr>';
echo '</thead>';
}

if ( $settings['rows'] ) {
echo '<tbody>';
$rows = $settings['rows'];
for ($i=0; $i < count($rows) - 1; $i++) {
Copy link
Owner

Choose a reason for hiding this comment

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

I think the rows minus 1 is causing a problem.

Elementor used always spaces in between, check the other files in the repository.

Suggested change
for ($i=0; $i < count($rows) - 1; $i++) {
for ( $i=0; $i < count( $rows ); $i++ ) {

Copy link
Author

Choose a reason for hiding this comment

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

yes I should have put <= like in the JS side

echo '<tr>';
echo '<td class="elementor-repeater-item-' . esc_attr( $rows[$i]['_id'] ) . '">' .$rows[$i]['column_content_one'] . '</td>';
echo '<td class="elementor-repeater-item-' . esc_attr( $rows[$i]['_id'] ) . '">' .$rows[$i]['column_content_two'] . '</td>';
echo '<td class="elementor-repeater-item-' . esc_attr( $rows[$i]['_id'] ) . '">' .$rows[$i]['column_content_three'] . '</td>';
echo '<tr>';
}

echo '</tbody>';
};
echo '</table>';
}


protected function content_template() {
?>

<table>
<# if ( 'yes' === settings.show_title ) { #>
<thead>
<tr>
<th>{{{ settings.column_one }}}</th>
<th>{{{ settings.column_two }}}</th>
<th>{{{ settings.column_three }}}</th>
</tr>
</thead>
<# } #>

<# if ( settings.rows.length ) { #>
<tbody>
<# let rows = settings.rows #>
<# for(let i = 0; i <= rows.length - 1; i++){ #>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<# for(let i = 0; i <= rows.length - 1; i++){ #>
<# for( let i = 0; i <= rows.length - 1; i++ ){ #>

Copy link
Author

Choose a reason for hiding this comment

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

i changed it

<tr>
<td class="elementor-repeater-item-{{ rows[i]._id }}">{{{rows[i].column_content_one}}}</td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td class="elementor-repeater-item-{{ rows[i]._id }}">{{{rows[i].column_content_one}}}</td>
<td class="elementor-repeater-item-{{ rows[i]._id }}">{{{ rows[i].column_content_one }}}</td>

Copy link
Author

Choose a reason for hiding this comment

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

I changed it

<td class="elementor-repeater-item-{{ rows[i]._id }}">{{{rows[i].column_content_two}}}</td>
<td class="elementor-repeater-item-{{ rows[i]._id }}">{{{rows[i].column_content_three}}}</td>
</tr>
<# } #>
</tbody>
<# } #>
</table>
<?php
}

}