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

www.credit-agricole.fr – “Open Hours“ exceed the allocated table space #6603

Closed
softvision-oana-arbuzov opened this issue May 10, 2017 · 5 comments

Comments

@softvision-oana-arbuzov
Copy link
Member

URL: https://www.credit-agricole.fr/spip.php?page=detail_agence&id_agence=6443&id_secteur_contexte=2455
Browser / Version: Firefox Mobile Nightly 55.0a1 (2017-05-09)
Operating System: Nexus 5 (Android 6.0.1) - Resolution 1080 x 1920 pixels (~445 ppi pixel density)
Problem type: Layout is messed up

Steps to Reproduce

  1. Navigate to: https://www.credit-agricole.fr/spip.php?page=detail_agence&id_agence=6443&id_secteur_contexte=2455
  2. Observe “Horaires D’Ouverture” (“Open Hours”) section.

Expected Behavior:
The hours fir in the allocated table space.

Actual Behavior:
The hours exceed the allocated table space.

Note:

  1. Reproducible on Firefox 52.0 Release
  2. Not reproducible on Chrome (Mobile) 57.0.2987.132.
  3. Screenshot attached.

Watchers:
@softvision-sergiulogigan
@softvision-oana-arbuzov

sv; country: fr

Screenshot Description

From webcompat.com with ❤️

@softvision-oana-arbuzov softvision-oana-arbuzov changed the title www.credit-agricole.fr - layout is messed up www.credwww.credit-agricole.fr – “Open Hours“ exceed the allocated table space May 10, 2017
@adamopenweb adamopenweb changed the title www.credwww.credit-agricole.fr – “Open Hours“ exceed the allocated table space www.credit-agricole.fr – “Open Hours“ exceed the allocated table space May 10, 2017
@MDTsai
Copy link

MDTsai commented May 11, 2017

The time is in a table. I think reason is by the default font size. Maybe add word-wrap could solve this issue.

@softvision-oana-arbuzov
Copy link
Member Author

The issue is still reproducible.

Affected area:

<div>
   <table>
      <tbody>
         <tr class="odd">
            <td class="jour">Lundi</td>
            <td>fermé</td>
         </tr>
         <tr class="even">
            <td class="jour">Mardi</td>
            <td>08h15-12h00|13h30-17h30</td>
         </tr>
         <tr class="odd">
            <td class="jour">Mercredi</td>
            <td>08h15-12h00|13h30-17h30</td>
         </tr>
         <tr class="even">
            <td class="jour">Jeudi</td>
            <td>08h15-12h00|13h30-15h45</td>
         </tr>
         <tr class="odd">
            <td class="jour">Vendredi</td>
            <td>08h15-12h00|13h30-17h30</td>
         </tr>
         <tr class="even">
            <td class="jour">Samedi</td>
            <td>08h15-12h15</td>
         </tr>
         <tr class="odd">
            <td class="jour">Dimanche</td>
            <td>fermé</td>
         </tr>
      </tbody>
   </table>
</div>

Modifying the font-size from default to .7500em, the text no longer exceeds the allocated space.

#pleine-page .agence-horaires table {
    font-size: .8571em;
}

Note that the issue is not reproducible on Google Pixel (Android 8.1.0) - 1080 x 1920 pixels (~441 ppi pixel density).

Tested with:
Browser / Version: Firefox Mobile Nightly 59.0a1 (2018-01-04)
Operating System: Samsung Galaxy S7 Edge (Android 7.0) - Resolution 1440 x 2560 pixels (~534 ppi pixel density)

@karlcow
Copy link
Member

karlcow commented Mar 26, 2018

Both Chrome and Firefox have for table in their browser stylesheet

word-break: normal
word-spacing: 0px
word-wrap: normal

I guess the difference seems to be about how - is handled in Chrome and in Firefox.

Firefox right
Chrome left

https://codepen.io/webcompat/pen/YareqB

Screenshot Description

Chrome allows to break after the - it means that it is a breakable character. It's very similar to this issue
https://bugzilla.mozilla.org/show_bug.cgi?id=1423408

@karlcow
Copy link
Member

karlcow commented Mar 26, 2018

Close as duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=1448776

@karlcow
Copy link
Member

karlcow commented Mar 26, 2018

I opened a new bug on bugzilla about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants