Hi Nathan,
On Mon, 4 Jan 2021, Nathan Sidwell wrote:
> Here's a patch describing the c++20 modules status. While there I
> noticed the coroutines project status was out of date (it's done).
>
> ok?
you don't need extra approval for release note entries in your
areas of expertise/maintainership, but I'm always happy to help
and have a look.
I skimmed these changes and they look good.
The one part that caught my attention is
- <td rowspan=10> Modules </td>
+ <td rowspan=10> Modules<br> </td>
where I'm wondering why you are adding the <br> here?
Is this to top-align that cell since other cells in that row have
multiple lines? If so, we should be able to generally handle this
via CSS. Check out gcc.css and the existing entry
table.cxxdrstatus td:nth-child(4) { text-align:center; }
there. If you want this for all rows, something like the following
should do what you want?
table.cxxdrstatus td:nth-child(1) { vertical-align:top; }
Or do you only want to do that for that one row? Then we could add
a new class.
Either way, if you don't want to dive in, got ahead and commit as is,
let me know what you want to accomplish, and I'll have a look. :-)
Gerald
