Skip to content

Remove official logo header image#12064

Merged
alecslupu merged 1 commit intodevelopfrom
remove/official_img_header
Nov 24, 2023
Merged

Remove official logo header image#12064
alecslupu merged 1 commit intodevelopfrom
remove/official_img_header

Conversation

@andreslucena
Copy link
Copy Markdown
Member

🎩 What? Why?

In the new design we're no longer using the "official header logo", so this PR removes it.

📌 Related Issues

Testing

All the specs should be green
There should not be any more reference to this field

📷 Screenshots

Screenshot of the decidim.barcelona homepage with the header logo circled

♥️ Thank you!

@andreslucena andreslucena added module: core module: admin type: removal PRs that implement a removal of a functionality or code labels Nov 23, 2023
github-actions[bot]
github-actions bot previously approved these changes Nov 23, 2023
@andreslucena andreslucena marked this pull request as ready for review November 24, 2023 10:01
@andreslucena
Copy link
Copy Markdown
Member Author

I'm not sure if we :

  1. Left some note in the Releases Notes, just in case someone wants to backup this file
  2. Have some protocol on how we handle these data / files removals. Something like not having the migration for removing the column database until the next version?

Copy link
Copy Markdown
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Tested locally, And everything went well. Pipeline is green.

I would change something in the layout appearance screen.

For a bit, i freaked out that This PR could have broken the Layout ... as the icon button is not so clearly visible ... This is something that is not in the scope of this PR. cc @andreslucena , @decidim/product

image

@alecslupu alecslupu merged commit ddbb7da into develop Nov 24, 2023
@alecslupu alecslupu deleted the remove/official_img_header branch November 24, 2023 12:16
entantoencuanto added a commit that referenced this pull request Nov 29, 2023
…/redesign-staging

* fix/control-comments-replies-polymorphism:
  Add a polymorphic option to restrict acts_as_tree lookup
  Add tests to check the 'Hide replies' message error
  Fix pipeline after translations were updated (#12084)
  Remove the duplicate reference of icons (#12083)
  Fix modal forms for editor (#12077)
  New Crowdin updates (#12003)
  Display only once resource type in search (#12041)
  Redesign: fix responsiveness on nav-bar admin (#12045)
  Apply flash styles to Announcements (part 2) (#12039)
  Remove legacy design support from WYSIWYG editor (TipTap) (#12015)
  Extract methods from meetings seeds (#12053)
  Show message when there are no content in the page (#12056)
  Show message when there are no sortitions (#12057)
  Fix icons on flash alerts (#12069)
  Update numbering in RELEASE_NOTES.md (#12066)
  Add prerequirements to the manual installation tutorial (#12068)
  Differentiate non-clickable elements in admin menu (#12000)
  Remove official logo header image (#12064)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: admin module: core type: removal PRs that implement a removal of a functionality or code

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Official logo header is no longer used

2 participants