#930 Fixes #752 - Make the TOC responsive for the order page

Merged
max.mehl merged 1 commits from vincent:order_toc_small_screen into master 4 weeks ago
vincent commented 1 month ago
There is no content yet.
vincent added this to the Hackathon1905 milestone 1 month ago
reinhard commented 1 month ago
Collaborator

Thank you Vincent! I think the PR fixes the actual issue and should be merged.

However, it does not fix the underlying root of the issue: that our standard CSS does not contain gernal, reusable classes for a table of contents, resulting in a number of pages constructing their individual TOC design. Let’s keep in mind that this is soemthing we should change.

Thank you Vincent! I think the PR fixes the actual issue and should be merged. However, it does not fix the underlying root of the issue: that our standard CSS does not contain gernal, reusable classes for a table of contents, resulting in a number of pages constructing their individual TOC design. Let's keep in mind that this is soemthing we should change.
vincent commented 1 month ago
Poster

I’d suggest to put this in style.less and make sure every TOC has the #toc id

#toc {
  float: right;
  clear: right;
  border: solid 1px;
  margin: .5em;
  background: rgb(240,240,240);
  width: 30%;
}

@media only screen and (max-width: 600px) {
  div#toc {
    width: 100%;
  }
}

Edit: Created an issue here

I'd suggest to put this in style.less and make sure every TOC has the `#toc` id ```css #toc { float: right; clear: right; border: solid 1px; margin: .5em; background: rgb(240,240,240); width: 30%; } @media only screen and (max-width: 600px) { div#toc { width: 100%; } } ``` Edit: Created an issue [here](https://git.fsfe.org/FSFE/fsfe-website/issues/942)
reinhard commented 1 month ago
Collaborator

In principle I agree with the above CSS, however I think it could be better integrated with some bootstrap features.

In principle I agree with the above CSS, however I think it could be better integrated with some bootstrap features.
max.mehl commented 1 month ago
Owner

@reinhard @vincent @mweimann Would you like to take another look at how to make it more bootstrap-native, or shall I merge for now to make it at least a bit better?

I tend toward coming up with a decent TOC layout for all kind of pages (and update those), so I’d love to have a quite optimal one

@reinhard @vincent @mweimann Would you like to take another look at how to make it more bootstrap-native, or shall I merge for now to make it at least a bit better? I tend toward coming up with a decent TOC layout for all kind of pages (and update those), so I'd love to have a quite optimal one
vincent commented 4 weeks ago
Poster

I think we should merge this and use #942 to explore possibilities for a global css class.

I think we should merge this and use [#942](https://git.fsfe.org/FSFE/fsfe-website/issues/942) to explore possibilities for a global css class.
reinhard commented 4 weeks ago
Collaborator

I agree with @vincent.

I agree with @vincent.
max.mehl commented 4 weeks ago
Owner

Fine for me. However, @vincent, could you please undo the changes for the da, fr, sq and tr version, or delete these files? They are currently outdated and should not be made up-to-date by these changes:

$ tools/check-translation-status.sh -a -f order/order.en.xhtml 
Basefile: order/order.en.xhtml ( 2019-05-22 )
  STATUS      LANG    DATE
  --------    ----   ----------
  OUTDATED     da    2017-06-13
  OUTDATED     fr    2017-06-13
  OUTDATED     sq    2017-06-13
  OUTDATED     tr    2018-09-06
  Up-to-date   de    2019-05-22
  Up-to-date   it    2019-05-22
  Up-to-date   nl    2019-05-22
Fine for me. However, @vincent, could you please undo the changes for the da, fr, sq and tr version, or delete these files? They are currently outdated and should not be made up-to-date by these changes: ``` $ tools/check-translation-status.sh -a -f order/order.en.xhtml Basefile: order/order.en.xhtml ( 2019-05-22 ) STATUS LANG DATE -------- ---- ---------- OUTDATED da 2017-06-13 OUTDATED fr 2017-06-13 OUTDATED sq 2017-06-13 OUTDATED tr 2018-09-06 Up-to-date de 2019-05-22 Up-to-date it 2019-05-22 Up-to-date nl 2019-05-22 ```
max.mehl added a new dependency 4 weeks ago
vincent commented 4 weeks ago
Poster

Done!

Done!
max.mehl commented 4 weeks ago
Owner

Excellent, thanks!

Excellent, thanks!
vincent deleted branch order_toc_small_screen 4 weeks ago
The pull request has been merged.
Sign in to join this conversation.
Loading…
Cancel
Save
There is no content yet.