Skip to content

Order property assignments#134

Merged
ScriptRaccoon merged 2 commits intomainfrom
order-property-assignments
Apr 25, 2026
Merged

Order property assignments#134
ScriptRaccoon merged 2 commits intomainfrom
order-property-assignments

Conversation

@ScriptRaccoon
Copy link
Copy Markdown
Owner

@ScriptRaccoon ScriptRaccoon commented Apr 25, 2026

When manually assigning properties to categories, it often happens that one proof refers to an assignment made earlier in the file. However, before this PR, manually assigned properties were listed alphabetically. This could be confusing, since the proofs could not be read from top to bottom in their natural order. It could also result in less important properties being listed first.

This PR resolves the issue by assigning an autoincrementing ID to every property assignment, whether manual or deduced. Previously, the table used a composite primary key consisting of category ID and property ID. Displayed properties are now ordered by ID and therefore appear in the same order as in the file where they are assigned. This also allows contributors to deliberately choose which properties should be shown first.

The position column previously used for deduced properties is now redundant. Deduced properties are still displayed in a logical order, unaffected by this PR, and their order cannot be influenced by contributors.

The same remarks apply to assignments of properties to functors.

Before

This is from the walking_fork page. It is weird that "cofiltered-limit-stable epimorphisms" appears first.

Bildschirmfoto 2026-04-25 um 13 02 54

After

Bildschirmfoto 2026-04-25 um 13 03 04

This has now the same order as in the file:

(
	'walking_fork',
	'finite',
	TRUE,
	'This is trivial.'
),
(
	'walking_fork',
	'small',
	TRUE,
	'This is trivial.'
),
...

Before

This is taken from the Top page. It is weird that "locally small" is quite at the bottom, while "coregular" is quite at the top (among other things).

Bildschirmfoto 2026-04-25 um 13 13 44

After

Bildschirmfoto 2026-04-25 um 13 13 53

@ScriptRaccoon ScriptRaccoon merged commit 2374cf2 into main Apr 25, 2026
1 check passed
@ScriptRaccoon ScriptRaccoon deleted the order-property-assignments branch April 25, 2026 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant