Contrib.social

move user.admin.css into seven where it belongs

Open on Drupal.org →
Created on 19 May 2015, over 9 years ago
Updated 30 January 2023, almost 2 years ago

Problem/Motivation

user.admin.css contains only css thats pure styling for the permission & account setting admin pages, that should be moved to Seven
None of the css is functional, - its pure styling - So no reason to have it hiding away down in a module folder

/* Permissions page */
.permissions .module {
  font-weight: bold;
}
.permissions .permission {
  padding-left: 1.5em; /* LTR */
}
[dir="rtl"] .permissions .permission {
  padding-left: 0;
  padding-right: 1.5em;
}

/* Account settings */
.user-admin-settings .details-description {
  font-size: 0.85em;
  padding-bottom: .5em;
}

Proposed resolution

Remaining tasks

delete user.admin.css
copy css into seven theme

User interface changes

none

API changes

📌 Task
Status

Needs work

Version

10.1 ✨

Component
User system  →

Last updated 1 day ago

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 screenshots

    The change alters the user interface, so before and after screenshots should be added to document the UI change. Make sure to capture the relevant region only. Use a tool such as Aviary on Windows or Skitch on Mac OS X.

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.

  • Comment almost 2 years ago →
    needs-review-queue-bot

    The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

    Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

    Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

  • Comment almost 2 years ago →
    🇮🇳India gauravvvv Delhi, India

    What are the next steps. As seven is no longer part of 10.1.x. I have added Needs issue summary update tag.

  • Merge request !8166moved user.admin.css into claro where it belongs → (Open) created by Unnamed author
  • Status changed to Needs review 8 months ago
  • Comment 8 months ago →
    🇮🇳India gauravvvv Delhi, India
  • Pipeline finished with Success
    8 months ago
    Total: 585s
    #180170
  • Status changed to Needs work 8 months ago
  • Comment 8 months ago →
    🇺🇸United States smustgrave

    Still needs issue summary updates and title

    Did not review.

  • Status changed to Needs review 8 months ago
  • Comment 8 months ago →
    🇮🇳India gauravvvv Delhi, India

    Updated issue summary and title

  • Status changed to Needs work 8 months ago
  • Comment 8 months ago →
    🇺🇸United States smustgrave

    Left a comment on the MR.

  • First commit to issue fork.
  • Comment 3 months ago →
    🇮🇳India vinmayiswamy

    Hi,
    I have added deprecation to the user.admin.css library following this guide: How to Deprecate → .
    I also created a Change Record → for the same.

    Kindly review the changes and please let me know if any further adjustments are needed. I appreciate any feedback or suggestions for improvements.

    Thanks!

  • Pipeline finished with Failed
    3 months ago
    Total: 3920s
    #310830
  • First commit to issue fork.
  • Pipeline finished with Failed
    3 months ago
    Total: 469s
    #313862
  • Pipeline finished with Failed
    3 months ago
    Total: 639s
    #313872
  • Pipeline finished with Failed
    3 months ago
    Total: 467s
    #316791
  • Pipeline finished with Failed
    3 months ago
    Total: 8113s
    #317326
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024