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

Achille/dev 0.0.25 #31

Merged
merged 2 commits into from
Nov 22, 2022
Merged

Achille/dev 0.0.25 #31

merged 2 commits into from
Nov 22, 2022

Conversation

AchilleBailly
Copy link
Collaborator

Version 0.0.25

  • Issue on @<whatever> when whatever was not the right format (compared to used regexp) : added the possibility to use space in between and handled to case when there is no match.
  • Small visual fix with fixed columns and keys.

th.dtfc-fixed-left {
z-index: 2;
}

/* loading */
.lds-dual-ring {
display: inline-block;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Makes sure that moving columns go behind fixed ones.

if (!$e->valid) {
continue;
}

$cls = trim(strtolower($e->{$classement}));

# selectionner tout ceux qui sont dans la db
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tests when the event is not valid. For now only used when there is no match on the place in the summary.

}


return $cls;
}

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Handling cases where the regexp does not match

}
}

export const optionDatatable1 = {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Two different options for datatables : optionDatatables2 when 2 fixed columns are needed, optionDatatables1 when there is only 1

@baimard baimard merged commit 7f55658 into main Nov 22, 2022
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.

2 participants