Skip to content

Feat/cells #296

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

Open
wants to merge 26 commits into
base: feat/clubs
Choose a base branch
from
Open

Feat/cells #296

wants to merge 26 commits into from

Conversation

Aryawart-kathpal
Copy link
Member

Initialized the ipr-cell page under the research tab.
Linked to issue #290

Copy link
Contributor

@aryansri-19 aryansri-19 left a comment

Choose a reason for hiding this comment

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

IKS and IIC are under institute/cells
IPR is under research/ipr

<div className="absolute left-1/2 top-1/2 -translate-x-1/2 -translate-y-1/2 text-center">
<h1
className="mx-2 my-auto text-xl md:text-2xl lg:text-3xl xl:text-4xl"
style={{ color: 'black' }}
Copy link
Contributor

Choose a reason for hiding this comment

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

use tailwind
text-neutral-900

Comment on lines 145 to 149
<TableRow>
<TableHead>Sr. No.</TableHead>
<TableHead>Responsibility</TableHead>
<TableHead>Name of Faculty</TableHead>
</TableRow>
Copy link
Contributor

Choose a reason for hiding this comment

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

For table headers, use translations

Comment on lines 177 to 180
<TableRow>
<TableHead>Sr. No.</TableHead>
<TableHead>Past activities</TableHead>
</TableRow>
Copy link
Contributor

Choose a reason for hiding this comment

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

Here too.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks fine, just use tailwind wherever used style

/>
<section className="w-full">
<div>
<Table scrollAreaClassName="h-96">
Copy link
Contributor

Choose a reason for hiding this comment

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

make table horizontally scrollable for smaller screens like mobile ( generally 648px )

@@ -12,7 +12,22 @@ export interface Translations {
more: string;
};
};

club: {
Copy link
Contributor

Choose a reason for hiding this comment

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

The key in translation should be Capitalized

Copy link
Member Author

Choose a reason for hiding this comment

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

I have not made this change, may be someone else had made this change previously

@Aryawart-kathpal
Copy link
Member Author

@aryansri-19 please review the changes

@Aryawart-kathpal Aryawart-kathpal changed the base branch from master to feat/clubs April 30, 2025 12:48
@aryansri-19
Copy link
Contributor

Been a while, let's see
why are there 18 files modified?!

@Aryawart-kathpal
Copy link
Member Author

@aryansri-19 See the commit stack above, these are not only my changes, it includes changes made by 7 people. My base branch is not master in this pr, but staged.

@aryansri-19
Copy link
Contributor

@aryansri-19 See the commit stack above, these are not only my changes, it includes changes made by 7 people. My base branch is not master in this pr, but staged.

but the changes you made are not in all those 18 files, right?
thoda hectic hoga but rebase your branch with the staged, and then commit your changes.

@Aryawart-kathpal
Copy link
Member Author

@aryansri-19 See the commit stack above, these are not only my changes, it includes changes made by 7 people. My base branch is not master in this pr, but staged.

but the changes you made are not in all those 18 files, right? thoda hectic hoga but rebase your branch with the staged, and then commit your changes.

Can't do it right now, because I am using some changes here that I had made in feat/cells..
Firstly it has to be merged with feat/cells

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.

5 participants