Skip to content

Redesign: integrate highlighted process and last activities in menu#11093

Merged
ferblape merged 17 commits intofeature/redesignfrom
feature/redesign-fix-highlighted-process
Jul 1, 2023
Merged

Redesign: integrate highlighted process and last activities in menu#11093
ferblape merged 17 commits intofeature/redesignfrom
feature/redesign-fix-highlighted-process

Conversation

@ferblape
Copy link
Copy Markdown
Contributor

@ferblape ferblape commented Jun 23, 2023

🎩 What? Why?

This PR completes the integration of the menu highlighted process and last activities.

📌 Related Issues

Testing

Check that:

  • if you prepare a highlighted process with top weight it should appear in the menu
  • the last activities are included

📷 Screenshots

Screenshot 2023-06-23 at 11 05 39

♥️ Thank you!

@ferblape ferblape changed the base branch from develop to feature/redesign June 23, 2023 10:38
@ferblape ferblape requested a review from furilo June 23, 2023 12:23
@ferblape ferblape added the project: redesign Barcelona City Council contract label Jun 23, 2023
@ferblape ferblape marked this pull request as ready for review June 23, 2023 12:23
@ferblape ferblape changed the title Feature/redesign fix highlighted process Redesign / Integrate highlighted process and last activities in menu Jun 23, 2023
@ferblape ferblape force-pushed the feature/redesign-fix-highlighted-process branch from de1099d to 376b7fb Compare June 23, 2023 12:24
@furilo
Copy link
Copy Markdown
Contributor

furilo commented Jun 23, 2023

@Crashillo there are a couple of details that should be reviewed:

  • If you hover an avatar, the author info appears below the dropdown
  • The width of the boxes in the top and in the bottom part should be equal, so that they are aligned.
image

Crashillo and others added 12 commits June 23, 2023 16:24
* small tweaks for process/assembly cards

* remove upcoming meeting from assembly card

* assembly card no longer show upcoming meeting
…ab (#11051)

* Display only public users followings in following profile tab

* Force use of redesigned profile cell

* Fix followings counters to consider only users and groups and fix officialization_text

* Use link for personal url in profile

* Remove deprecated examples

* Change expected texts

* Adapt tests of follows

* Add REDESIGN_PENDING skip

* Add cell spec for redesigned_profile_cell

* Fix spelling

* Adapt tests to redesigned profile

* Remove deprecated examples

* Prepare profile actions buttons to use options

* Add missing resend email confirmation instructions button

* Display user_profile_bottom hook in redesigned profile cell

* Make profile_holder redesigned profile cell public available from hook

* Change styles of member_of user_profile partial

* Recover test

* simplify markup

* add separator

* Fix typo

Co-authored-by: Fran Bolívar <francisco.bolivar@nazaries.com>

---------

Co-authored-by: Hugoren Martinako <aumpfbahn@gmail.com>
Co-authored-by: Fernando Blat <fernando@blat.es>
Co-authored-by: Fran Bolívar <francisco.bolivar@nazaries.com>
* simplify layout center

* use cols-8 layout for public profile

* span to 10 cols

* Simplify code

* Restore conditional

* Revert "Restore conditional"

This reverts commit a7661bb.

* Revert "Simplify code"

This reverts commit a3db3dc.

* Add clarification

* Use case to check conditions

* Offense

---------

Co-authored-by: Fernando Blat <fernando@blat.es>
* Use the component name in the sidebar title

* Control when current_component is not defined

* Test component name in the sidebar

* Discard conferences behaviour
* Update package lock

* Update package lock in design app
@furilo furilo requested a review from a team June 27, 2023 08:08
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.

LGTM, thanks @ferblape

@ferblape ferblape requested a review from fblupi June 28, 2023 09:51
@furilo
Copy link
Copy Markdown
Contributor

furilo commented Jun 29, 2023

@entantoencuanto a last detail: I've realized that if you are in a space and open the Home/General Dropdown, the Last Activity is the same as the one of the Space, not of the whole site

image

@andreslucena andreslucena changed the title Redesign / Integrate highlighted process and last activities in menu Redesign: integrate highlighted process and last activities in menu Jun 29, 2023
@furilo
Copy link
Copy Markdown
Contributor

furilo commented Jun 29, 2023

@fblupi apart from the last comment I added, this can be reviewed.

Copy link
Copy Markdown
Member

@fblupi fblupi left a comment

Choose a reason for hiding this comment

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

Just a suggestion to fix the error @furilo pointed out.

Please, fix the conflicts too.

entantoencuanto and others added 2 commits June 30, 2023 11:49
…_breadcrumb_main_dropdown.html.erb

Co-authored-by: Fran Bolívar <francisco.bolivar@nazaries.com>
…-process

* feature/redesign:
  Redesign: display online meeting (#11075)
  Redesign: home menu content block (#11048)
  Redesign: home (#10920)
  Remove widgets and embeds (#11096)
  Redesign / Complete card G cell specs (#11094)
  Redesign: processes assemblies metadata content blocks (#10637)
  Redesign: vertical margin inside the callouts (#11071)
  Redesign / Update package-lock  (#11091)
  Use the component name in the sidebar title (#11088)
  Redesign: layout center (#11068)
  Redesign: display only public users followings in following profile tab (#11051)
  Redesign: fix cards (#11072)
@ferblape ferblape merged commit 7c90732 into feature/redesign Jul 1, 2023
@ferblape ferblape deleted the feature/redesign-fix-highlighted-process branch July 1, 2023 04:26
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.

6 participants