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

Created on 6 February 2015, over 9 years ago
Updated 3 June 2024, 25 days 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 about 16 hours 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.69.0 2024