Skip to content
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

Changes for Jitsi UI #518

Merged
merged 5 commits into from
Jul 19, 2020
Merged

Conversation

nisarg1499
Copy link
Contributor

Issue Solved : #516
Image of UI after changes
jitsi2
Allignment is done and height issue is also solved.

Please review : @jaskiratsingh2000 @vaibhavdaren

@jaskiratsingh2000
Copy link
Member

LGTM!

@AuraOfDivinity
Copy link
Collaborator

@jaskiratsingh2000 @vaibhavdaren A small concern from my end, Is this the suggested button format that we are planning to use within the other Modals used throughout the platform?

@devesh-verma
Copy link
Member

@jaskiratsingh2000 @vaibhavdaren A small concern from my end, Is this the suggested button format that we are planning to use within the other Modals used throughout the platform?

Nope. It should match with other button designs

Copy link
Member

@Rupeshiya Rupeshiya left a comment

Choose a reason for hiding this comment

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

@nisarg1499 Please use the css classes which is used in all the other modals.
Just have a look at modals.scss.
You don't need to write css for this. Also for this part why you modified comments.scss ??
It would have effect the comment modals as well. Please look at it.

@nisarg1499
Copy link
Contributor Author

@nisarg1499 Please use the css classes which is used in all the other modals.
Just have a look at modals.scss.
You don't need to write css for this. Also for this part why you modified comments.scss ??
It would have effect the comment modals as well. Please look at it.

Actually the css of comment file was being used in join call button. But now I have updated the css of jitsi and now it's independent. And the design element is also same as others.
Output after changes :
new-jitsi

src/user/integrations/NameForm.js Outdated Show resolved Hide resolved
src/user/integrations/NameForm.js Outdated Show resolved Hide resolved
@nisarg1499
Copy link
Contributor Author

Final UI.
finalll

@Rupeshiya
Copy link
Member

Rupeshiya commented Jul 16, 2020

Final UI.
finalll

Looks fine now.

@vaibhavdaren vaibhavdaren merged commit b9c51d8 into codeuino:development Jul 19, 2020
vaibhavdaren added a commit that referenced this pull request Aug 6, 2020
* Implements markdown and priview functionality

* Parsing content within comment

* Comment notifications (#501)

* Initial

* Addition of comments section

* requested changes

* Changes for Jitsi UI (#518)

* css changes for jitsi box ui change

* edited jitsi.scss for jitsi ui and reverted changes of comment.scss

* changes made for making same fonts

* added modals in jitsi ui

* removed unnecessary code from jitsi.scss

* fixing dev branch issues (#525)

* implemented user activity and fix inconsistencies (#527)

* Settings integrate and deploy configuration (#530)

* integrated settings page and user activity modification

* configure to deploy

* Insights (#529)

* Adding reactGA

* Insights section

* removing /hompage path

* Requested changes

* initial changes

* Fetching data from the backend and integrating redux

* Fixing modal issues (#531)

* social links fix (#545)

* fix issues (#555)

* Fixing proposal functionality issues (#553)

Co-authored-by: Asel <peirisasel373@gmail.com>
Co-authored-by: Nisarg Shah <nisshah1499@gmail.com>
Co-authored-by: Rupesh Krishna Jha <rupeshiya@gmail.com>
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.

None yet

6 participants