From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hamza.pair.com (hamza.pair.com [209.68.5.143]) by sourceware.org (Postfix) with ESMTPS id 023F43861834 for ; Mon, 4 Jan 2021 20:33:14 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 023F43861834 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=pfeifer.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gerald@pfeifer.com Received: from hamza.pair.com (localhost [127.0.0.1]) by hamza.pair.com (Postfix) with ESMTP id 7E53133E25; Mon, 4 Jan 2021 15:33:13 -0500 (EST) Received: from [192.168.1.5] (77.118.226.34.wireless.dyn.drei.com [77.118.226.34]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by hamza.pair.com (Postfix) with ESMTPSA id BF3F133E24; Mon, 4 Jan 2021 15:33:12 -0500 (EST) Date: Mon, 4 Jan 2021 21:33:06 +0100 (CET) From: Gerald Pfeifer To: Nathan Sidwell cc: gcc-patches@gcc.gnu.org Subject: Re: cxx status update In-Reply-To: Message-ID: <105a4d0-3311-f5ab-e09e-f4ce95ad394@pfeifer.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Spam-Status: No, score=-3.5 required=5.0 tests=BAYES_20, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 04 Jan 2021 20:33:15 -0000 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 - Modules + Modules
where I'm wondering why you are adding the
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