Skip to content

Redesign: omnipresent banner#10847

Merged
ferblape merged 5 commits intofeature/redesignfrom
feature/redesign-omnipresent-banner
May 23, 2023
Merged

Redesign: omnipresent banner#10847
ferblape merged 5 commits intofeature/redesignfrom
feature/redesign-omnipresent-banner

Conversation

@Crashillo
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Redesign version of the omnipresent banner

📌 Related Issues

📷 Screenshots

https://decidim-redesign.populate.tools/

@Crashillo Crashillo added the project: redesign Barcelona City Council contract label May 11, 2023
@Crashillo Crashillo requested review from ferblape and furilo May 11, 2023 15:55
@furilo
Copy link
Copy Markdown
Contributor

furilo commented May 12, 2023

  • Reduce the height a little bit
  • All area should be clickable
  • Hover: Transition + Apply the same color as to buttons
  • Remove the text-underline on hover

@furilo
Copy link
Copy Markdown
Contributor

furilo commented May 12, 2023

@decidim/product take a look in case you have more/other comments.

@furilo
Copy link
Copy Markdown
Contributor

furilo commented May 12, 2023

(Btw, @decidim/product: I'm not 100% comfortable with using that color - but can't come up with any solid alternative, thus I would proceed with this and think on alternatives during our upcoming design-details-cycle)

@Crashillo Crashillo linked an issue May 12, 2023 that may be closed by this pull request
@carolromero
Copy link
Copy Markdown
Member

@furilo I'm not comfortable with this solution either. At the moment I'd make the following changes and we can see what other ideas come out in the last sprint.:

  • Reduce banner height to half the height of the menu.
  • Use footer background color and adjust font size.

image

To give you all the context, we're going to make this banner dismissible in the next iteration.

@furilo
Copy link
Copy Markdown
Contributor

furilo commented May 15, 2023

OK, @Crashillo will adjust spacing.

What about using a variation of the primary color? Something like 20% for the background.

@Crashillo
Copy link
Copy Markdown
Contributor Author

I left the footer color background and the black on hover (as the second footer)

@Crashillo Crashillo requested a review from a team May 16, 2023 09:34
Copy link
Copy Markdown
Member

@carolromero carolromero left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks @Crashillo!

@ferblape ferblape requested a review from a team May 17, 2023 05:11
* feature/redesign:
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
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.

LGTM
image

@ferblape ferblape merged commit e630e34 into feature/redesign May 23, 2023
@ferblape ferblape deleted the feature/redesign-omnipresent-banner branch May 23, 2023 02:33
entantoencuanto added a commit that referenced this pull request May 26, 2023
* feature/redesign:
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
entantoencuanto added a commit that referenced this pull request May 26, 2023
…rafts

* feature/redesign:
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
  simplify 2col layout (#10819)
  Redesign: processes groups content blocks (#10491)
  Redesign: assemblies content blocks (#10573)
entantoencuanto added a commit that referenced this pull request May 26, 2023
* feature/redesign: (21 commits)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
  simplify 2col layout (#10819)
  Redesign: processes groups content blocks (#10491)
  Redesign: assemblies content blocks (#10573)
  Redesign: process & process group cards (#10716)
  Redesign: conference cards (#10502)
  Redesign: pending blogs (#10686)
  Redesign: blog cards (#10685)
  Redesign: filters (#10390)
  replace uses of specific margin-bottom for layout margins (#10675)
  Redesign: menu mobile (#10351)
  Fix comments scss to avoid compilation errors (#10657)
  ...
entantoencuanto added a commit that referenced this pull request May 26, 2023
* feature/redesign:
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
entantoencuanto added a commit that referenced this pull request Jun 2, 2023
…blies-details-page

* feature/redesign:
  Redesign: amendments (#10765)
  Redesign: proposals (#10555)
  Remove unused preset-env dependencies (#10916)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
entantoencuanto added a commit that referenced this pull request Jun 2, 2023
* feature/redesign:
  Redesign: amendments (#10765)
  Redesign: proposals (#10555)
  Remove unused preset-env dependencies (#10916)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
entantoencuanto added a commit that referenced this pull request Jun 2, 2023
* feature/redesign:
  Redesign: amendments (#10765)
  Redesign: proposals (#10555)
  Remove unused preset-env dependencies (#10916)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
entantoencuanto added a commit that referenced this pull request Jun 2, 2023
* feature/redesign:
  Redesign: amendments (#10765)
  Redesign: proposals (#10555)
  Remove unused preset-env dependencies (#10916)
  Feature/redesign components breadcrumb (#10441)
  Redesign: debates (#10653)
  Redesign: Remove reference to commentable in activity cell (#10816)
  Redesign: spinner (#10848)
  Redesign: omnipresent banner (#10847)
  Add redesign enable ENV variable to control the pipeline (#10610)
  Redesign: progress bar (#10638)
  Redesign: meeting cards (#10722)
  Redesign: pending login (#10699)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

project: redesign Barcelona City Council contract

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Redesign / Include CSS for 'omnipresent-banner'

6 participants