Skip to content
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

[FIX] website_slides: fix various UI issues with comments/likes #30027

Closed
wants to merge 2 commits into from

Conversation

awa-odoo
Copy link
Contributor

@awa-odoo awa-odoo commented Jan 8, 2019

Task #1923443

Purpose

This commit fixes several problems with the website UI of the slides module:

  1. Probably due to the migration to bootstrap 4, the active tab panel in the 'slide_detail_view' was not displayed on page loading (and only when switching back and forth in the tabs).

  2. When adding a like/dislike to a slide, the like/dislike counter was not properly updated.

closes #29901

Copy link
Contributor

@tde-banana-odoo tde-banana-odoo left a comment

Choose a reason for hiding this comment

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

Zizisse typo !

@@ -462,7 +462,7 @@
tabindex="0"
data-toggle="popover">
<i class="fa fa-thumbs-up fa-1x" role="img" aria-label="Likes" title="Likes"/>
<t t-esc="slide.likes"/>
<span class="o_wslides_like_dislike_count" t-esc="slide.likes"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

wslides ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

According to my secret informant in the website office, the new name convention for classes is something like:

  • o (for le Odoo)
  • w (for le website)
  • slides (for le module)

He gave me the following example: o_wevent_registration_list

Copy link
Contributor

Choose a reason for hiding this comment

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

cross_eye

@C3POdoo C3POdoo added the RD research & development, internal work label Jan 8, 2019
@awa-odoo awa-odoo changed the title [FIX] website_slides: fix various UI issues with comments/downloads/likes [FIX] website_slides: fix various UI issues with comments/likes Jan 9, 2019
@tde-banana-odoo
Copy link
Contributor

@robodoo r+

@robodoo robodoo added the r+ 👌 label Jan 9, 2019
@tde-banana-odoo
Copy link
Contributor

@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Jan 9, 2019

Merge method set to rebase and fast-forward

@robodoo
Copy link
Contributor

robodoo commented Jan 9, 2019

I'm sorry, @tde-banana-odoo. This PR is already reviewed, reviewing it again is useless.

@tde-banana-odoo
Copy link
Contributor

@robodoo r+ retry redo do stufff please I am alone in the drak don't try to ff vive les frites

@robodoo
Copy link
Contributor

robodoo commented Jan 9, 2019

I'm sorry, @tde-banana-odoo. Retry makes no sense when the PR is not in error. This PR is already reviewed, reviewing it again is useless.

@tde-banana-odoo
Copy link
Contributor

@robodoo retry

@robodoo
Copy link
Contributor

robodoo commented Jan 10, 2019

I'm sorry, @tde-banana-odoo. Retry makes no sense when the PR is not in error.

@tde-banana-odoo
Copy link
Contributor

@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Jan 10, 2019

Merge method set to rebase and fast-forward

@robodoo
Copy link
Contributor

robodoo commented Jan 10, 2019

I'm sorry, @tde-banana-odoo. This PR is already reviewed, reviewing it again is useless.

Task #1923443

Purpose
=======

Probably due to the migration to bootstrap 4, the active tab panel in the 'slide_detail_view' was not displayed
on page loading (and only when switching back and forth in the tabs).

partly closes odoo#29901
Task #1923443

Purpose
=======

When adding a like/dislike to a slide, the like/dislike counter was not properly updated.

partly closes odoo#29901
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed r+ 👌 labels Jan 10, 2019
@awa-odoo
Copy link
Contributor Author

@robodoo retry

@robodoo
Copy link
Contributor

robodoo commented Jan 11, 2019

I'm sorry, @awa-odoo. Retry makes no sense when the PR is not in error.

@tde-banana-odoo
Copy link
Contributor

@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Jan 11, 2019

Merge method set to rebase and fast-forward

robodoo pushed a commit that referenced this pull request Jan 11, 2019
Task #1923443

Purpose
=======

When adding a like/dislike to a slide, the like/dislike counter was not properly updated.

partly closes #29901

closes #30027
@robodoo
Copy link
Contributor

robodoo commented Jan 11, 2019

Merged, thanks!

@robodoo robodoo closed this Jan 11, 2019
@fw-bot fw-bot deleted the 12.0-fix-website-slide-awa branch October 20, 2019 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[12] website: Comments are not displayed on Slides
4 participants