Skip to content

[New Admin] Update category views.#1148

Merged
lastpotion merged 4 commits intorevamp_adminfrom
update/categories
Mar 22, 2017
Merged

[New Admin] Update category views.#1148
lastpotion merged 4 commits intorevamp_adminfrom
update/categories

Conversation

@lastpotion
Copy link
Copy Markdown
Contributor

@lastpotion lastpotion commented Mar 20, 2017

🎩 What? Why?

Update categories section inside a process with the new admin views and styles.

📌 Related Issues

📋 Subtasks

  • Update views
  • Fix tests

📷 Screenshots (optional)

screen shot 2017-03-21 at 11 19 41
screen shot 2017-03-21 at 11 19 48

👻 GIF

@lastpotion lastpotion self-assigned this Mar 20, 2017
@codecov-io
Copy link
Copy Markdown

codecov-io commented Mar 21, 2017

Codecov Report

Merging #1148 into revamp_admin will decrease coverage by 0.02%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           revamp_admin    #1148      +/-   ##
================================================
- Coverage         97.11%   97.09%   -0.03%     
================================================
  Files               422      422              
  Lines              7048     7048              
================================================
- Hits               6845     6843       -2     
- Misses              203      205       +2
Impacted Files Coverage Δ
...controllers/decidim/admin/categories_controller.rb 88% <0%> (-4%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0e2e2df...cd0e3d1. Read the comment docs.

</td>
<td class="table-list__actions">
<% if can? :update, category %>
<%= link_to edit_participatory_process_category_path(participatory_process, category), class: "action-icon action-icon--edit", data: { tooltip: true, disable_hover: false }, title: t("actions.edit", scope: "decidim.admin") do %>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use @Dor3nz's helper everywhere

@mention-bot
Copy link
Copy Markdown

@lastpotion, thanks for your PR! By analyzing the history of the files in this pull request, we identified @oriolgual and @mrcasals to be potential reviewers.

@lastpotion lastpotion merged commit 47a1202 into revamp_admin Mar 22, 2017
@lastpotion lastpotion deleted the update/categories branch March 22, 2017 13:20
beagleknight pushed a commit that referenced this pull request Mar 27, 2017
* Add category views

* Fix tests

* Add tests

* Add new icon_helper
beagleknight pushed a commit that referenced this pull request Mar 28, 2017
* Add category views

* Fix tests

* Add tests

* Add new icon_helper
beagleknight pushed a commit that referenced this pull request Mar 29, 2017
* Add category views

* Fix tests

* Add tests

* Add new icon_helper
beagleknight pushed a commit that referenced this pull request Mar 30, 2017
* Add admin styles

* Add basic layout

* Change scopes icon

* Add participatory processes index page

* Add participatory process layout

* Fix some specs

* Fix more broken tests

* Fix more specs

* Fix poltergeist

* New admin - Settings section (#1124)

* Add participatory process layout

* Add basic layout for settings

* Finish settings form

* Add scopes index page

* Finish basic scopes admin

* Add revamp_admin to travis

* Fix a few specs

* Fix broken tests

* Remove bad merge

* [New admin] - Pages section (#1130)

* Add index section

* Add new section

* Finish forms

* Fix specs

* [New admin] - Users (#1136)

* Add layout

* Add users index page

* Add invite user form

* Add user groups index and fix pagination for both users and user groups

* Fix table links

* Fix specs

* Fix admin invite user labels

* [New admin] - Newsletters (#1140)

* Add layout and index page

* Add new/edit forms

* Add newsletter preview

* Fix specs

* [New Admin] - Process groups (#1141)

* Add new views

* Fix broken specs

* [New Admin] - Style process features (#1143)

* Style process features

* Refactor feature tests for admin

* [New Admin] - Processes info / steps (#1138)

* updated new process view

* updated process steps index

* update process steps index table

* update processes layouts

* Processes steps views updated

* Fix indentation

* Remove unusued locales

* Update tests

* Add moderations missing menu

* missing td added

* Add icon link helper (#1155)

* Add icon link helper

* Update existing icon links

* Add optional data to icon link helper

* Add methods at icon links

* Refactor tests for new icon helper

* Add docs and refactor helper call

* Refactor all icon links with new helper call

* [New Admin] Update category views. (#1148)

* Add category views

* Fix tests

* Add tests

* Add new icon_helper

* [New Admin] Attachment update. (#1158)

* Views added

* Finish tests

* add helper

* Style process features at admin (#1149)

* [New Admin] - Process groups (#1141)

* Add new views

* Fix broken specs

* Style meetings table

* Update branch with icon helper

* Style proposals

* Style results

* Style budgets

* Update missing styles for some forms

* Fix most of the tests

* Fix more tests

* Style moderation tables (#1182)

* Remove unused travis step

* [New Admin] Update process admins views. (#1183)

* Add new and edit forms

* clean update command

* Tests added

* Amend test

* Add dashboard content

* Spellcheck

* Amend managing roles strings

* Use user instead of admin

* Fix participatory process edit button

* Add some feedback

* Add styles for feature forms

* Add edit feature form

* Add more feedback

* [New Admin] Html and styles review. (#1200)

* Review all the forms

* Class coherence review

* CA: Swap funcionalitat with component

* Fix a few specs

* ES: Swap funcionalidad with componente

* Fix a few more specs

* Add missing helper

* Add preview images to organization settings form

* Icons updated on features table

* amend icon
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants