merge node.*.css files into node.admin.css

Created on 6 February 2015, almost 10 years ago
Updated 18 September 2024, 4 months ago

Problem/Motivation

The node module has 3 different css files (node.admin.css, node.module.css, node.preview.css) that are all related to admin tasks.

Proposed resolution

Merge the 3 files into node.admin.css and remove any duplicated code.

Remaining tasks

1. Resolve questions raised in #35 โ†’ @idbr

The issue summary indicates all 3 nodes.*.css files are merged into a single node.admin.css, but the patch leaves node.module.css. Is this on purpose?

node.admin.theme.css contains box-sizing, float, width attributes that deliver layout. This seems inconsistent with the SMACCS policy. Can you include a comment why these attributes are in a .theme file instead of a .module file?

User interface changes

Nil.


API changes

Nil.

Beta phase evaluation

<!--Uncomment the relevant rows for the issue. -->
๐Ÿ“Œ Task
Status

Needs work

Version

11.0 ๐Ÿ”ฅ

Component
Node systemย  โ†’

Last updated 5 days ago

No maintainer
Created by

๐Ÿ‡ฉ๐Ÿ‡ฐDenmark mortendk

Live updates comments and jobs are added and updated live.
  • CSS

    It involves the content or handling of Cascading Style Sheets.

  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024