-
Notifications
You must be signed in to change notification settings - Fork 136
Feature/project listing updates #1331
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
— Initial commit with basic fields added to the app project type
— Added the new fields to app_dev project template. Deploying feature branch to dev for testing.
— Added desired UI, however, it has a problem that it shows the first value as selected value of the slider radio. Working to fix that.
— Made first value of the slide-radiogroup type questions so that SliderInput component’s bug is not noticeable.
— Fixed issue with default value of Slider component
…dates#1268 Feature/project listing updates#1268
…/project-listing-updates # Conflicts: # circle.yml
…/project-listing-updates
— Added extended questions to all templates
— centre align the cards
— Hidden description, as per suggestions by vic, for 3 of new fields added.
— Updated notes section description as per suggestion. It is done for all project templates except mobility and real world testing because those two have different copy.
— Relocated card components to their proper place and fixed css issue.
— Spacing between cards should be 20px horizontally and vertically
-- Space between project icon and project title/project date is 10px
— Project title is the wrong size/weight — project timestamp (created) is missing
— Project description should be auto-sizing to always fit text - minimum 1 line, maximum 8 lines, cut above 8 lines with ellipsis.
— Project status isn't spelling the "paused" status
— Project status label doesn't match design (we need a new style definition) - text size and text color;
— Project members should be spaced with 5px between avatar image
— Project card on hover should be a link to open the project - the whole card should be clickable;
— Remove the shadow on "create new project" card;
…dates#1268 Feature/project listing updates#1268
…isting-updates#1268
…dates#1268 Feature/project listing updates#1268
…appirio-tech/connect-app into feature/project-listing-updates
— Fixed as requested
…eing but still need find the root cause.
…roject intake forms — Remove budget estimate section from Website specification page
…ge project status from any state to any state including active state even without joining project.
1. It does not load ProjectStatus in dropdown enhanced mode which prevent attaching unnecessary dropdown events 2. it does not fire onClickOutside method always even if ProjecStatus is editable and rendered as dropdown. Now it fires only when dropdown is open.
— Added infinite scroll to the customer’s view. It was necessary to load projects in paged manner because API does not support loading all project at once. API has hard limit of 20 for loading projects. As of Infinite scroll is causing two load calls when user scrolls to bottom, hence we might need to go with a simple load more button.
…isting-updates#1268
— Removed InfiniteScroll for time being as it is having issue of double page load. Added standard load more button instead to load more pages.
…dates#1268 Feature/project listing updates#1268
— Fixed the case when user selects, ‘Create New Project’ from the incomplete project screen.
— put a label "end of list" so people don't keep hitting the button
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.